A while back I asked for some feedback about how we should format anonymous methods.  Then I was asking whether the default should be to put the opening brace on the same line as the delegate keyword or not.  From your feedback, it looks like we'll keep the default the same as it is now (on the next line).  So that brings me to another question.  Some people have requested the ability to have the anonymous method braces indented.  I'll give some examples:

Currently, anonymous methods look something like this:

      List<Thing> somethings = things.FindAll(delegate(Thing thing)

      { return thing.ShouldBeIncluded; }

      );

 

Or this:

 

      List<Thing> somethings = things.FindAll(delegate(Thing thing)

      { return thing.ShouldBeIncluded; });

 

Or this:

 

      List<Thing> somethings = things.FindAll(delegate(Thing thing)

      {

            return thing.ShouldBeIncluded;

      });

 

Depending on where you put newlines.

 

With the proposed option, you would be able to make those look like:

 

      List<Thing> somethings = things.FindAll(delegate(Thing thing)

            { return thing.ShouldBeIncluded; }

      );

 

Or this:

 

      List<Thing> somethings = things.FindAll(delegate(Thing thing)

            { return thing.ShouldBeIncluded; });

 

Or this:

 

      List<Thing> somethings = things.FindAll(delegate(Thing thing)

            {

                  return thing.ShouldBeIncluded;

            });

 

Or even this:

 

      List<Thing> somethings = things.FindAll(delegate(Thing thing)

            {

                  return thing.ShouldBeIncluded;

            }

      );

 

 

So my question for you today is: Do you want the new option? If you do, what should the default be, indented, or not indented?

 

Please post and let me know.