alossar/treesharp

Automatically exported from code.google.com/p/treesharp

treesharp's Issues

Switch missing return and falls through to default.

The following lines of code that set the selection inside the switch statement 
will always fall through to assigning the default selection if there is one, 
even when the switch is found successfully.  There needs to be a return as 
noted below.

Switch.cs: lines 81-

            if (Arguments != null)
            {
                // Make sure we don't do this query twice.
                SwitchArgument<T> arg = Arguments.First(a => a.RequiredValue.Equals(value));
                if (arg != null)
                {
                    Selection = arg.Branch;
                    // NEEDS return statement added here "return;"
                }
            }

            if (Default != null)
            {
                Selection = Default;
            }

Original issue reported on code.google.com by [email protected] on 30 Nov 2012 at 12:52

DecoratorContinue

There seems to be a typo in the DecoratorContinue class, the first line of the 
Execute() function should be:
if (CanRun(context))

originally this statement is negated (!) which reverses the logic of the 
CanRunDecoratorDelegate.

Original issue reported on code.google.com by [email protected] on 10 Jul 2011 at 11:15

Contact info

Hi!

Can I contact you? Send me e-mail to mail[at]artemkoval[dot]com.

Best regards, Artem Koval.

Original issue reported on code.google.com by [email protected] on 2 May 2011 at 7:27