forked from torvalds/linux
-
Notifications
You must be signed in to change notification settings - Fork 488
Trailing comma in fmt! macro #1227
Copy link
Copy link
Closed
Labels
good first issueGood for newcomersGood for newcomersmediumExpected to be an issue of medium difficulty to resolve.Expected to be an issue of medium difficulty to resolve.• libRelated to the `rust/` library.Related to the `rust/` library.
Metadata
Metadata
Assignees
Labels
good first issueGood for newcomersGood for newcomersmediumExpected to be an issue of medium difficulty to resolve.Expected to be an issue of medium difficulty to resolve.• libRelated to the `rust/` library.Related to the `rust/` library.
This code:
results in this error:
Because of the trailing comma after
info.to_pid. This should be fixed so that trailing commas are allowed. I think the problem is in thefmt!proc macro, which is defined inrust/macros/fmt.rs.This requires submitting a proper patch to the LKML and the Rust for Linux mailing list. Please recall to test your changes (including generating the documentation if changed, running the Rust doctests if changed, etc.), to use a proper title for the commit, to sign your commit under the Developer's Certificate of Origin and to add a
Suggested-by:tag, and aLink:tag to this issue. Please see https://docs.kernel.org/process/submitting-patches.html and https://rust-for-linux.com/contributing for details.