src: use correct way to signal interceptor error

[targos]

Refs:

node/deps/v8/include/v8-function-callback.h

Lines 398 to 408 in e4f61de

	void ReturnValue<T>::Set(const Local<S> handle) {
	// "V8_DEPRECATE_SOON" this method if |T| is |void|.
	#ifdef V8_IMMINENT_DEPRECATION_WARNINGS
	static constexpr bool is_allowed_void = false;
	static_assert(!std::is_void<T>::value,
	"ReturnValue<void>::Set(const Local<S>) is deprecated. "
	"Do nothing to indicate that the operation succeeded or use "
	"SetFalse() to indicate that the operation failed (don't "
	"forget to handle info.ShouldThrowOnError()). "
	"See http://crbug.com/348660658 for details.");
	#else

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 87.32%. Comparing base (e4f61de) to head (b196b6c).
Report is 50 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff             @@
##             main   #54418      +/-   ##
==========================================
- Coverage   87.33%   87.32%   -0.01%     
==========================================
  Files         648      648              
  Lines      182321   182321              
  Branches    34971    34973       +2     
==========================================
- Hits       159222   159217       -5     
- Misses      16374    16377       +3     
- Partials     6725     6727       +2     

Files	Coverage Δ
src/node_webstorage.cc	73.64% <100.00%> (ø)

... and 17 files with indirect coverage changes

[nodejs-github-bot]

CI: https://ci.nodejs.org/job/node-test-pull-request/61182/

[nodejs-github-bot]

CI: https://ci.nodejs.org/job/node-test-pull-request/61404/

[nodejs-github-bot]

CI: https://ci.nodejs.org/job/node-test-pull-request/61408/

[nodejs-github-bot]

CI: https://ci.nodejs.org/job/node-test-pull-request/61411/

[nodejs-github-bot]

Landed in c062b52