[PATCH] D58095: [clang-tidy] Make google-objc-function-naming ignore implicit functions πŸ™ˆ

Stephane Moore via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Feb 19 20:43:23 PST 2019


stephanemoore marked 4 inline comments as done.
stephanemoore added inline comments.


================
Comment at: clang-tools-extra/test/clang-tidy/Inputs/Headers/stdio.h:1
+#ifndef _STDIO_H_
+#define _STDIO_H_
----------------
aaron.ballman wrote:
> aaron.ballman wrote:
> > stephanemoore wrote:
> > > stephanemoore wrote:
> > > > I noticed that some of the other example headers don't have `#ifdef` guards at all. I decided to still include themβ€”perhaps mostly for my own sanity. I figured that it wasn't necessary to use the full `LLVM_CLANG_TOOLS_EXTRA_TEST_CLANG_TIDY_INPUTS_HEADER_STDIO_H` for this header that is mimicking a system header so I went with the shorter `_STDIO_H_` which I think better represents what actual stdio.h headers would have. Let me know if you think something else makes more sense.
> > > Do I need the licensing preamble on this test input header?
> > Yes, please add it.
> I don't think it matters all that much for our tests; I'm fine with the way you have it currently.
Added the licensing preamble. Let me know if anything looks amiss.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D58095/new/

https://reviews.llvm.org/D58095





More information about the cfe-commits mailing list