[PATCH] D53882: [clang-tidy] Adding Zircon checker for std namespace

Eugene Zelenko via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Oct 31 10:54:01 PDT 2018


Eugene.Zelenko added inline comments.


================
Comment at: clang-tools-extra/docs/ReleaseNotes.rst:176
+
+  Warns when the `std` namespace is used, as its use is against Zircon's libc++
+  policy for the kernel.
----------------
JonasToth wrote:
> alexfh wrote:
> > JonasToth wrote:
> > > s/its/it's/
> > > 
> > > Could `std` be considered code here? Not sure, but maybe using quotes is better?
> > Actually, "its" is correct in this context ("its use" vs "it's used").
> whupsi. sry for noise.
Please synchronize with first statement in documentation.


https://reviews.llvm.org/D53882





More information about the cfe-commits mailing list