Please do not reply to this email- if you want to comment on the bug, go to the URL shown below and enter your comments there.
Changed by [EMAIL PROTECTED] http://bugzilla.ximian.com/show_bug.cgi?id=82462 --- shadow/82462 2007-08-16 19:21:32.000000000 -0400 +++ shadow/82462.tmp.32332 2007-08-16 19:22:24.000000000 -0400 @@ -1,14 +1,14 @@ Bug#: 82462 Product: Mono: Class Libraries Version: unspecified -OS: +OS: unknown OS Details: Status: NEW Resolution: -Severity: +Severity: Unknown Priority: Minor Component: Sys.Core AssignedTo: [EMAIL PROTECTED] ReportedBy: [EMAIL PROTECTED] QAContact: [EMAIL PROTECTED] TargetMilestone: --- @@ -20,6 +20,17 @@ (it's obvious in the code). ------- Additional Comments From [EMAIL PROTECTED] 2007-08-16 19:21 ------- Created an attachment (id=19825) A patch for the issue. + +------- Additional Comments From [EMAIL PROTECTED] 2007-08-16 19:22 ------- +Here's a patch. Should I commit? + +I've rewritten the method more efficiently by manually writing the +"foreach", to avoid having an "if" within the loop. This does not IMO +make the code any less readable, and actually makes it shorter. + +I've used 4-spaces for indentation rather than tabs, to conform with +the existing code (which consistently violates the Mono Coding +Guidelines in several ways!). _______________________________________________ mono-bugs maillist - [email protected] http://lists.ximian.com/mailman/listinfo/mono-bugs
