[clang-tools-extra] r316650 - [clangd] Don't use /// for non-doxygen comments.

Benjamin Kramer via cfe-commits cfe-commits at lists.llvm.org
Thu Oct 26 06:41:15 PDT 2017


On Thu, Oct 26, 2017 at 3:16 PM, Jonas Hahnfeld <hahnjo at hahnjo.de> wrote:
> Am 2017-10-26 06:07, schrieb Benjamin Kramer via cfe-commits:
>>
>> Author: d0k
>> Date: Thu Oct 26 03:07:04 2017
>> New Revision: 316650
>>
>> URL: http://llvm.org/viewvc/llvm-project?rev=316650&view=rev
>> Log:
>> [clangd] Don't use /// for non-doxygen comments.
>>
>> Modified:
>>     clang-tools-extra/trunk/clangd/fuzzer/ClangdFuzzer.cpp
>>     clang-tools-extra/trunk/clangd/tool/ClangdMain.cpp
>>
>> Modified: clang-tools-extra/trunk/clangd/fuzzer/ClangdFuzzer.cpp
>> URL:
>>
>> http://llvm.org/viewvc/llvm-project/clang-tools-extra/trunk/clangd/fuzzer/ClangdFuzzer.cpp?rev=316650&r1=316649&r2=316650&view=diff
>>
>> ==============================================================================
>> --- clang-tools-extra/trunk/clangd/fuzzer/ClangdFuzzer.cpp (original)
>> +++ clang-tools-extra/trunk/clangd/fuzzer/ClangdFuzzer.cpp Thu Oct 26
>> 03:07:04 2017
>> @@ -14,16 +14,12 @@
>>
>>
>> //===----------------------------------------------------------------------===//
>>
>>  #include "ClangdLSPServer.h"
>> -#include "llvm/Support/Program.h"
>>  #include <sstream>
>>
>>  extern "C" int LLVMFuzzerTestOneInput(uint8_t *data, size_t size) {
>> -  /// Change stdin to binary to not lose \r\n on windows.
>> -  llvm::sys::ChangeStdinToBinary();
>> -
>
>
> Did you want to remove this statement?

Yeah, it's not needed. Thanks for pointing it out, the change
should've been in the previous commit but I somehow messed this up.


More information about the cfe-commits mailing list