On 02.05.2013 5:10, Jordan Rose wrote:

On May 1, 2013, at 18:08 , Anton Yartsev <[email protected] <mailto:[email protected]>> wrote:

On 01.05.2013 21:10, Jordan Rose wrote:

On May 1, 2013, at 6:38 , Anton Yartsev <[email protected] <mailto:[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.
Updated at r180901. Did this helped?


Jordan

--
Anton

_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits

Reply via email to