851584 - DOM bindings example generator generates two Length() methods for array like interfaces

Status: RESOLVED FIXED

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

Description

[Johnny Stenback (:jst)]

Given the WebIDL:

interface MimeTypeArray {
  readonly attribute unsigned long length;

  getter MimeType? item(unsigned long index);
  getter MimeType? namedItem(DOMString name);
};

I get this output in the example header:

  uint32_t Length() const;

  nsMimeType* Item(uint32_t index);

  nsMimeType* NamedItem(const nsAString& name);

  nsMimeType* IndexedGetter(uint32_t index, bool &found);

  nsMimeType* NamedGetter(const nsAString& name, bool &found);

  uint32_t Length();

  void GetSupportedNames(nsTArray< nsString >& retval);

which of course is one Length() method too many.

Comment 1

[Boris Zbarsky [:bzbarsky]]

Attachment: Don't generate a Length() on our C++ example class to handle our indexed getter if we already have a 'length' attribute in the WebIDL.

Comment 2

[Boris Zbarsky [:bzbarsky]]

http://hg.mozilla.org/integration/mozilla-inbound/rev/13fb55ee93bb

Comment 3

[Ed Morley [:emorley]]

https://hg.mozilla.org/mozilla-central/rev/13fb55ee93bb