Hello Gábor, I was thinking that a discussion is dead. It seems like I have fixed all the issues that were pointed with this review, but there were no replies. So, I was thinking that nobody needs this functionality. You can try this patch and leave your opinion about it if you have any suggestions. About long command lines... AFAIR command line is a common way to pass parameters for all parts of Clang, including CSA. Anyway, there are only few checkers that may support this option now so it cannot add a lot to the command line currently.
> Hi Aleksei! > > What is the state of this patch? Do you still working on it? I am very > interested in this feature. > > One minor note though: there are some hard limit on some platforms on the > length of the command line (as far as I know it's about 8k characters on > windows. source: http://support.microsoft.com/kb/830473). Command line also > contains include paths and several other valuable information. At Ericsson we > can have command lines that are several thousand characters long when > invoking the static analyzer. Adding configuration options to the command > line might result in hitting the character limit. In the long run I think it > would be better to support configuration files. > > http://reviews.llvm.org/D3967 > > EMAIL PREFERENCES > > http://reviews.llvm.org/settings/panel/emailpreferences/ http://reviews.llvm.org/D3967 EMAIL PREFERENCES http://reviews.llvm.org/settings/panel/emailpreferences/ _______________________________________________ cfe-commits mailing list [email protected] http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
