[cfe-commits] r139367 - /cfe/trunk/lib/Sema/AnalysisBasedWarnings.cpp

Caitlin Sadowski supertri at google.com
Mon Sep 12 15:30:12 PDT 2011


>> +using namespace thread_safety;
>
> Is this necessary? I would generally prefer not to have a proliferation of
> using namespace declarations. This namespace in particular seems not
> unlikely to have names that collide with others in use in this file (that
> is, other namespaces within Analysis).

Note this is an intermediate patch. This line is no longer in
AnalysisBasedWarnings.cpp

>> +namespace clang { namespace thread_safety {
>
> This seems a bit unconventional. I would just use two lines.

Patch committed.

Cheers,

Caitlin



More information about the cfe-commits mailing list