On Apr 16, 2013, at 14:44 , Ted Kremenek <[email protected]> wrote:
> +/// the allocatin site (where possible). Typo here. Separating analyzer config options out into different files does make sense. The reason I had hesitated to do it was because of the cached options (Optional<Bool>), but several of those could be redesigned in the same way as what you've done for RetainCountChecker, moving them to be local instance variables. A finite number of string parses for each ExprEngine isn't so bad. No immediate call to action here, but it would be nice to have a plan to stop bloating AnalyzerOptions the way I have been. Jordan
_______________________________________________ cfe-commits mailing list [email protected] http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
