-
Clemens Hammacher authored
Cpplint usually checks for non-const reference arguments. They are forbidden in the style guide, and v8 does not explicitly make an exception here. This CL re-enables that warning, and fixes all current violations by adding an explicit "NOLINT(runtime/references)" comment. In follow-up CLs, we should aim to remove as many of them as possible. TBR=mlippautz@chromium.org Bug: v8:9429 Change-Id: If7054d0b366138b731972ed5d4e304b5ac8423bb Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/1687891Reviewed-by: Clemens Hammacher <clemensh@chromium.org> Reviewed-by: Michael Lippautz <mlippautz@chromium.org> Reviewed-by: Jakob Kummerow <jkummerow@chromium.org> Reviewed-by: Igor Sheludko <ishell@chromium.org> Commit-Queue: Clemens Hammacher <clemensh@chromium.org> Cr-Commit-Position: refs/heads/master@{#62551}
8e11cc39
Name |
Last commit
|
Last update |
---|---|---|
.. | ||
OWNERS | ||
async-hooks-wrapper.cc | ||
async-hooks-wrapper.h | ||
d8-console.cc | ||
d8-console.h | ||
d8-js.cc | ||
d8-platforms.cc | ||
d8-platforms.h | ||
d8-posix.cc | ||
d8-windows.cc | ||
d8.cc | ||
d8.h |