777074 - dictionary_helper_gen.py doesn't handle 'unsigned long' types

Status: RESOLVED FIXED

(Core :: XPConnect, defect)

Description

[Mike Habicher [:mikeh] (high bugzilla latency)]

Steps to reproduce:
1. edit an IDL to include a dictionary with 'unsigned long' types, e.g.:
dictionary Foo {
  unsigned long bar;
};
2. modify dictionary_helper_gen.conf to include your IDL;
3. make

Expected behaviour: should handle unsigned long types.

Observed behaviour:
Traceback (most recent call last):
  File "/home/mikeh/dev/mozilla/btg011/gecko/config/pythonpath.py", line 56, in <module>
    main(sys.argv[1:])
  File "/home/mikeh/dev/mozilla/btg011/gecko/config/pythonpath.py", line 48, in main
    execfile(script, frozenglobals)
  File "/home/mikeh/dev/mozilla/btg011/gecko/js/xpconnect/src/dictionary_helper_gen.py", line 464, in <module>
    print_cpp_file(outfd, conf)
  File "/home/mikeh/dev/mozilla/btg011/gecko/js/xpconnect/src/dictionary_helper_gen.py", line 247, in print_cpp_file
    print_cpp(idl, fd, conf, d[0], dicts)
  File "/home/mikeh/dev/mozilla/btg011/gecko/js/xpconnect/src/dictionary_helper_gen.py", line 188, in print_cpp
    write_cpp(p, fd)
  File "/home/mikeh/dev/mozilla/btg011/gecko/js/xpconnect/src/dictionary_helper_gen.py", line 401, in write_cpp
    write_getter(a, iface, fd)
  File "/home/mikeh/dev/mozilla/btg011/gecko/js/xpconnect/src/dictionary_helper_gen.py", line 341, in write_getter
    raise BaseException("Unsupported type %s found in dictionary %s" % (realtype, iface.name))
BaseException: Unsupported type PRUint32  found in dictionary Dimensions
make[7]: *** [DictionaryHelpers.cpp] Error 1
make[7]: *** Deleting file `DictionaryHelpers.cpp'
make[6]: *** [libs] Error 2
make[5]: *** [libs_tier_platform] Error 2
make[4]: *** [tier_platform] Error 2
make[3]: *** [default] Error 2
make[2]: *** [realbuild] Error 2
make[1]: *** [build] Error 2

Note: signed 'long' works fine.

Comment 1

[Olli Pettay [:smaug][[e-mail]]]

Attachment: patch

Comment 2

[Olli Pettay [:smaug][[e-mail]]]

Attachment: patch

Comment 3

[Olli Pettay [:smaug][[e-mail]]]

Btw, this stuff used to work when dictionaries used XPCOM stuff.

Comment 4

[Olli Pettay [:smaug][[e-mail]]]

Attachment: with commit message

At least it compiles :)

Comment 5

[Mike Habicher [:mikeh] (high bugzilla latency)]

Compiles and runs as expected.

Comment 6

[Masatoshi Kimura [:emk]]

> +    elif realtype.count("PRUint32"):
> +        fd.write("    uint32_t u;\n")
> +        fd.write("    NS_ENSURE_STATE(JS_ValueToECMAUint32(aCx, v, &u));\n")
> +        fd.write("    aDict.%s = u;\n" % a.name)
You can pass |aDict.%s| directly to JS_ValueToECMAUint32 instead of using a temporary variable.
https://mxr.mozilla.org/mozilla-central/source/js/xpconnect/src/XPCConvert.cpp?rev=0581a34dde2a#412

Comment 7

[Olli Pettay [:smaug][[e-mail]]]

emk, want to upload such patch? I could review it immediately ;)

Comment 8

[Masatoshi Kimura [:emk]]

Sorry, I'm about to go out. Plaese create a patch yourself.

Comment 9

[Olli Pettay [:smaug][[e-mail]]]

Attachment: patch

Comment 10

[Olli Pettay [:smaug][[e-mail]]]

Comment on attachment 645561 [details] [diff] [review]
patch

too hard to get this 2 line patch right :/

Comment 11

[Olli Pettay [:smaug][[e-mail]]]

Attachment: patch

Comment 12

[Olli Pettay [:smaug][[e-mail]]]

https://hg.mozilla.org/mozilla-central/rev/9792ee3e5244

Comment 13

[Christoph Kerschbaumer [:ckerschb]]

Attachment: idl file using dictionaries

Comment 14

[Olli Pettay [:smaug][[e-mail]]]

Comment on attachment 645936 [details] [diff] [review]
idl file using dictionaries

Wrong bug?

Comment 15

[Gregor Wagner [:gwagner]]

Comment on attachment 645936 [details] [diff] [review]
idl file using dictionaries

interns :)

Comment 16

[Christoph Kerschbaumer [:ckerschb]]

wrong bug, sorry about that