Page MenuHomeFreeBSD

Remove the need for backslashes in syscalls.master.
ClosedPublic

Authored by brooks on Oct 9 2018, 10:04 PM.
Tags
None
Referenced Files
Unknown Object (File)
Thu, May 2, 8:28 AM
Unknown Object (File)
Thu, May 2, 8:27 AM
Unknown Object (File)
Thu, May 2, 7:18 AM
Unknown Object (File)
Thu, May 2, 12:39 AM
Unknown Object (File)
Thu, May 2, 12:17 AM
Unknown Object (File)
Wed, May 1, 10:52 PM
Unknown Object (File)
Feb 9 2024, 10:09 AM
Unknown Object (File)
Jan 13 2024, 11:28 AM
Subscribers
None

Details

Summary

Join non-special lines together until we hit a line containing a '}'
character. This allows the function declaration body to be split
across multiple lines without backslash continuation characters.

Continue to join lines ending with backslashes to allow gradual
migration and to support out-of-tree syscall vectors.

Test Plan

verified no changes in generated files

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

emaste added inline comments.
sys/kern/makesyscalls.sh
81 ↗(On Diff #48953)

we join with a space now?

This revision is now accepted and ready to land.Oct 10 2018, 12:43 AM

Might be it is a good idea to check that there is no nesting '(' or unbalanced ')' ?

brooks marked an inline comment as done.
  • Join without spaceso (previous behavior).
This revision now requires review to proceed.Oct 10 2018, 3:25 PM
  • Join without spaceso (previous behavior).

I didn't have a strong opinion on this either way, just wanted to make sure it was an intentional change.

This revision is now accepted and ready to land.Oct 10 2018, 3:26 PM
In D17488#373351, @kib wrote:

Might be it is a good idea to check that there is no nesting '(' or unbalanced ')' ?

The awk line parser makes sure things are of the form:

{<return_type> <function>(<arg_list>);}

I suppose one could check that <arg_list> contains balanced parentheses, but that seems beyond the scope of this change.

  • Join without spaceso (previous behavior).

I didn't have a strong opinion on this either way, just wanted to make sure it was an intentional change.

I didn't have a strong opinion and when I tried writing down my motivation for the change it sounded weak...

  • Allow both types of joins to be mixed. This makes it easier to semi-programatically convert files where alternative names are wrapped with backslashes.
This revision now requires review to proceed.Oct 15 2018, 8:07 PM
This revision is now accepted and ready to land.Oct 15 2018, 9:25 PM
This revision was automatically updated to reflect the committed changes.