[cfe-dev] Checker Plugins

Ted Kremenek kremenek at apple.com
Tue Aug 16 12:44:29 PDT 2011


On Aug 10, 2011, at 11:22 PM, Jordy Rose wrote:

> 
> On Aug 10, 2011, at 17:40, Argyrios Kyrtzidis wrote:
> 
>> On Aug 6, 2011, at 5:15 PM, Jordy Rose wrote:
>> 
>>> I was using "load" to mean "collect information about what checkers are available". Is there a better name for CheckerRegistry that captures the sense of "what checkers are available"? (Go back to "CheckerProvider"?)
>> 
>> Across llvm the "registry" concept is well established, I think we should follow the same term for "registering" available checkers and when actually creating concrete checker objects that the CheckerManager will use we can say this is "loading" or "initializing" or something else..
>> 
>> -Argyrios
> 
> 
> 
> All right, sounds good. I'll use "initialization". (And all the individual registerXChecker functions will become initializeXChecker in a separate patch.)
> 
> Jordy

Jordy,

I'm happy with these changes (coupled with Argiris's feedback).  I'm think it's time to start putting them into mainline.

Thanks for working on this.

Ted



More information about the cfe-dev mailing list