On May 1, 2013, at 18:08 , Anton Yartsev <[email protected]> wrote:
> On 01.05.2013 21:10, Jordan Rose wrote: >> >> On May 1, 2013, at 6:38 , Anton Yartsev <[email protected]> wrote: >> >>> On 01.05.2013 6:29, Jordan Rose wrote: >>>> Comments: >>>> >>>>> +use English; >>>> >>>> According to http://perldoc.perl.org/English.html, it's probably better to >>>> use >>>> >>>> use English qw( -no_match_vars ) ; >>>> >>>> ...not that scan-build's startup time matters that much. >>> Currently this is used only for $OSNAME while short names are used for all >>> other special perl variables. >>> Considering that using English affects performance, what about rolling back >>> to $^O and adding explanatory comment instead? >> >> Ah, right, fair enough. >> >> This looks good. Please go ahead and commit, but make sure that c++-analyzer >> is marked as executable for Unix-y systems. > Committed at r180900. Attributes of c++-analyzer are: -rwxr-xr-x Um, doesn't seem to be working for me. You may have to manually set svn:executable. Jordan
_______________________________________________ cfe-commits mailing list [email protected] http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
