diff options
| author | Lenny Maiorani <lenny@colorado.edu> | 2020-12-20 16:09:48 -0700 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2020-12-21 00:09:48 +0100 |
| commit | 765936ebaedfaa3a339d99a9865b555ddd7c23e2 (patch) | |
| tree | e092294ec99ca5b3ba9c92139f847dcd9a8a20bc /Libraries/LibWeb/Bindings | |
| parent | 4421d98e30763424055eefe729c9cab28abdf19d (diff) | |
| download | serenity-765936ebaedfaa3a339d99a9865b555ddd7c23e2.zip | |
Everywhere: Switch from (void) to [[maybe_unused]] (#4473)
Problem:
- `(void)` simply casts the expression to void. This is understood to
indicate that it is ignored, but this is really a compiler trick to
get the compiler to not generate a warning.
Solution:
- Use the `[[maybe_unused]]` attribute to indicate the value is unused.
Note:
- Functions taking a `(void)` argument list have also been changed to
`()` because this is not needed and shows up in the same grep
command.
Diffstat (limited to 'Libraries/LibWeb/Bindings')
| -rw-r--r-- | Libraries/LibWeb/Bindings/WindowObject.cpp | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/Libraries/LibWeb/Bindings/WindowObject.cpp b/Libraries/LibWeb/Bindings/WindowObject.cpp index 58b2045134..f147a86bf0 100644 --- a/Libraries/LibWeb/Bindings/WindowObject.cpp +++ b/Libraries/LibWeb/Bindings/WindowObject.cpp @@ -337,7 +337,6 @@ JS_DEFINE_NATIVE_GETTER(WindowObject::document_getter) JS_DEFINE_NATIVE_SETTER(WindowObject::document_setter) { // FIXME: Figure out what we should do here. Just ignore attempts to set window.document for now. - UNUSED_PARAM(value); } JS_DEFINE_NATIVE_GETTER(WindowObject::performance_getter) |