Skip to content

src: fix unused-result warning

Rodrigo Muino Tomonari requested to merge github/fork/santigimeno/fix_warning into master
Checklist
  • make -j4 test (UNIX), or vcbuild test nosign (Windows) passes
  • commit message follows commit guidelines
Affected core subsystem(s)

c++

Description of change

I was getting these warnings on OS X locally:

../src/node_util.cc:138:43: warning: ignoring return value of function declared with warn_unused_result attribute [-Wunused-result]
    PER_ISOLATE_PRIVATE_SYMBOL_PROPERTIES(V)
                                          ^~
../src/env.h:56:3: note: expanded from macro 'PER_ISOLATE_PRIVATE_SYMBOL_PROPERTIES'
  V(alpn_buffer_private_symbol, "node:alpnBuffer")                            \
  ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
../src/node_util.cc:133:3: note: expanded from macro 'V'
  target->Set(context,                                                        \
  ^~~~~~~~~~~ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
../src/node_util.cc:138:43: warning: ignoring return value of function declared with warn_unused_result attribute [-Wunused-result]
    PER_ISOLATE_PRIVATE_SYMBOL_PROPERTIES(V)
                                          ^~
../src/env.h:57:3: note: expanded from macro 'PER_ISOLATE_PRIVATE_SYMBOL_PROPERTIES'
  V(arrow_message_private_symbol, "node:arrowMessage")                        \
  ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
../src/node_util.cc:133:3: note: expanded from macro 'V'
  target->Set(context,                                                        \

Merge request reports