[Version 9.0] Feature support for attributes on local functions#1466
[Version 9.0] Feature support for attributes on local functions#1466BillWagner wants to merge 13 commits into
Conversation
e32a191 to
4184054
Compare
0fc9ff9 to
8b61c71
Compare
|
I reviewed the existing edits, made a couple small wording changes. This is ready for committee review for V9. |
8b61c71 to
6006520
Compare
655d59b to
28c46c5
Compare
|
@jskeet I've addressed all open comments on this one. It is small, and Joseph approved. I'd like to put this on the next agenda for possibly merging it. |
jskeet
left a comment
There was a problem hiding this comment.
Generally looks ready to go.
ab0c7b8 to
9039bb8
Compare
9039bb8 to
641fcc6
Compare
Add support for attributes and extern on local functions fix md formatting
This is now ready for committee review.
Co-authored-by: Joseph Musser <me@jnm2.com>
There were a few additional changes necessary for attributes on local functions.
641fcc6 to
bc9ef6c
Compare
|
An earlier version of this feature is already present on |
|
@jskeet If we finish the C# 8 work in time, this should be ready to merge for C# 9. |
|
|
||
| > *Example*: Consider the following: | ||
| > | ||
| > <!-- Example: {template:"standalone-console", name:"CallerMemberName1", inferOutput:true} --> |
There was a problem hiding this comment.
Can we easily extend the example to include a lambda? Presumably this would do the trick, if we've got Action declared elsewhere:
Action anonymousFunction = () => F2();
anonymousFunction();There was a problem hiding this comment.
addressed in the next commit.
addresses #1466 (review) extend the example to include a lambda expression expansion.
|
@jskeet If you're happy with the updated sample, this should be ready to go. |
Update the C# standard for the feature of applying attributes on local functions.
Note that this also includes specifying
externon a static local function.This contains all commits from #994. Subsequent edits also made.