1148506 - TestCodeGenBinding.cpp:10424:10: warning: unused variable 'result' [-Wunused-variable], in DOMProxyHandler::delete_

Status: RESOLVED FIXED

(Core :: DOM: Core & HTML, defect)

Description

[Daniel Holbert [:dholbert]]

Noticed two "unused variable" build warnings, for generated file TestCodeGenBinding.cpp:

obj/dom/bindings/TestCodeGenBinding.cpp:10424:10: warning: unused variable 'result' [-Wunused-variable]
     bool result;
          ^
..and...

obj/dom/bindings/TestCodeGenBinding.cpp:43418:10: warning: unused variable 'result' [-Wunused-variable]
     bool result;
          ^

(Same warning, just different line numbers.)

Both warnings are in a function with this function-signature:
bool
DOMProxyHandler::delete_(JSContext* cx, JS::Handle<JSObject*> proxy, JS::Handle<jsid> id, JS::ObjectOpResult& opresult) const

Comment 1

[Daniel Holbert [:dholbert]]

The first warning is pointing to code that looks like this:
====
  if (IsArrayIndex(index)) {
    bool deleteSucceeded;
    bool result;
    bool found = false;
    mozilla::dom::TestIndexedDeleterWithRetvalInterface* self = UnwrapProxy(proxy);
    deleteSucceeded = self->IndexedDeleter(index, found);
    MOZ_ASSERT(!JS_IsExceptionPending(cx));
    if (!found) {
      deleteSucceeded = true;
    }
    return deleteSucceeded ? opresult.succeed() : opresult.failCantDelete();
  }

  return dom::DOMProxyHandler::delete_(cx, proxy, id, opresult);
====

As you can see, "result" is declared & then never used.

Comment 2

[Boris Zbarsky [:bzbarsky]]

This became dead code in bug 1113369 part 5, where we started using deleteSucceeded for the operation result.  Should be able to just replace the

  decls = "bool result;\n"

bit in getDeleterBody with decls = "".

Comment 3

[Daniel Holbert [:dholbert]]

Attachment: fix v1

This just does what comment 2 suggests.

Comment 4

[Daniel Holbert [:dholbert]]

(I verified that this doesn't break my build, and the warning goes away.)

Comment 5

[Jason Orendorff [:jorendorff]]

Comment on attachment 8593010 [details] [diff] [review]
fix v1

Review of attachment 8593010 [details] [diff] [review]:
-----------------------------------------------------------------

Thank you.

Comment 6

[Pulsebot]

https://hg.mozilla.org/integration/mozilla-inbound/rev/b67384fd7f83

Comment 7

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/mozilla-central/rev/b67384fd7f83