[PATCH] D44294: [clangd] Fix diagnostic errors in the test code, NFC.

Ilya Biryukov via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Mar 12 07:48:48 PDT 2018


ilya-biryukov accepted this revision.
ilya-biryukov added inline comments.
This revision is now accepted and ready to land.


================
Comment at: unittests/clangd/XRefsTests.cpp:587
 
-  const char *HeaderContents = R"cpp([[]]int a;)cpp";
+  const char *HeaderContents = R"cpp([[]]#ifndef TEST_H_
+                                     #define TEST_H_
----------------
hokein wrote:
> ilya-biryukov wrote:
> > I would go with `#pragma once` in the test code, it adds less noise. WDYT?
> I thought `#pragma once` is non-standard, but the diagnostic message also suggest this fix. Changed to it.
Thanks! It's non-standard, but supported on every modern compiler, so we should be fine :-)


Repository:
  rCTE Clang Tools Extra

https://reviews.llvm.org/D44294





More information about the cfe-commits mailing list