Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

imp(arg_enum!): allows using meta items like repr(C) with arg_enum!s #544

Merged
merged 1 commit into from
Jun 28, 2016

Conversation

kbknapp
Copy link
Member

@kbknapp kbknapp commented Jun 28, 2016

One can now use more than one meta item, and things like #[repr(C)]

Example:

arg_enum! {
    #[repr(C)]
    #[derive(Debug)]
    pub enum MyEnum {
        A=1,
        B=2
    }
}

Closes #543

One can now use more than one meta item, and things like `#[repr(C)]`

Example:

```rust

arg_enum! {
	#[repr(C)]
	#[derive(Debug)]
	pub enum MyEnum {
		A=1,
		B=2
	}
}

```

Closes #543
@yo-bot
Copy link

yo-bot commented Jun 28, 2016

r? @sru

(yo-bot has picked a reviewer for you, use r? to override)

@coveralls
Copy link

Coverage Status

Coverage decreased (-0.02%) to 84.691% when pulling edf9b23 on issue-543 into 18237f4 on master.

@kbknapp
Copy link
Member Author

kbknapp commented Jun 28, 2016

@homu r+

@homu
Copy link
Contributor

homu commented Jun 28, 2016

📌 Commit edf9b23 has been approved by kbknapp

homu added a commit that referenced this pull request Jun 28, 2016
imp(arg_enum!): allows using meta items like repr(C) with arg_enum!s

One can now use more than one meta item, and things like `#[repr(C)]`

Example:

```rust

arg_enum! {
	#[repr(C)]
	#[derive(Debug)]
	pub enum MyEnum {
		A=1,
		B=2
	}
}

```

Closes #543
homu added a commit that referenced this pull request Jun 28, 2016
imp(arg_enum!): allows using meta items like repr(C) with arg_enum!s

One can now use more than one meta item, and things like `#[repr(C)]`

Example:

```rust

arg_enum! {
	#[repr(C)]
	#[derive(Debug)]
	pub enum MyEnum {
		A=1,
		B=2
	}
}

```

Closes #543
@homu
Copy link
Contributor

homu commented Jun 28, 2016

⌛ Testing commit edf9b23 with merge 3d5d865...

@homu
Copy link
Contributor

homu commented Jun 28, 2016

☀️ Test successful - status

@homu homu merged commit edf9b23 into master Jun 28, 2016
@kbknapp kbknapp deleted the issue-543 branch June 29, 2016 18:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants