[libcxx-commits] [PATCH] D110794: [libc++][test] NFC: Don't include name in banner comment

[Arthur O'Dwyer via Phabricator via libcxx-commits]

Quuxplusone requested changes to this revision.
Quuxplusone added a comment.
This revision now requires changes to proceed.

@jloser, I propose that I commandeer this PR to make the following completely mechanical changes in two separate and unreviewed commits:

- Normalize the copyright notice (by cutting and pasting from a good file), for all files in `git grep '//===-----' libcxx/test/ | grep -v '//===----------------------------------------------------------------------===//'`
- Remove `// -*- C++ -*-` from the first line of `find libcxx/test/ -name *.cpp` (but //not// from any other files)



================
Comment at: libcxx/test/support/constexpr_char_traits.h:1
-// -*- C++ -*-
-//===-------------------- constexpr_char_traits ---------------------------===//
----------------
Mordante wrote:
> I prefer to keep these in the header files .h can be a c or c++ file.
> IMO removing them has little benefit and might break somebodies setup.
FWIW, I //do// prefer to remove them from .h files to reduce cruft. I don't think anyone will work on libc++ very long in an editor that (1) cares what language the code is written in, and yet (2) can't be configured to open .h files as C++ by default.
However, I don't care to submit such a PR myself, and don't care enough to complain if someone else (e.g. you :)) blocks it.  And I agree that it should be done in a separate (completely mechanical) commit. If other people want to go have the political discussion, and decide, and come back and tell me "OK, go remove all the editor comments from `find libcxx/ -name *.h`," I //do// volunteer to make that mechanical commit myself. :)


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D110794