Last Comment Bug 775845 - Fail codegen on variadic arguments until they're implemented
: Fail codegen on variadic arguments until they're implemented
Status: RESOLVED FIXED
:
Product: Core
Classification: Components
Component: DOM (show other bugs)
: unspecified
: x86 Mac OS X
: -- normal (vote)
: mozilla17
Assigned To: Boris Zbarsky [:bz]
:
Mentors:
Depends on:
Blocks: ParisBindings
  Show dependency treegraph
 
Reported: 2012-07-19 21:40 PDT by Boris Zbarsky [:bz]
Modified: 2012-07-27 08:15 PDT (History)
2 users (show)
bzbarsky: in‑testsuite?
See Also:
Crash Signature:
(edit)
QA Whiteboard:
Iteration: ---
Points: ---
Has Regression Range: ---
Has STR: ---


Attachments
Fail codegen for variadic arguments until we add support for them. (1.22 KB, patch)
2012-07-19 21:42 PDT, Boris Zbarsky [:bz]
no flags Details | Diff | Splinter Review
Fail codegen for variadic arguments until we add support for them. (1.83 KB, patch)
2012-07-20 00:56 PDT, Boris Zbarsky [:bz]
khuey: review+
Details | Diff | Splinter Review

Description Boris Zbarsky [:bz] 2012-07-19 21:40:58 PDT
Right now we just silently treat them as non-variadic.
Comment 1 Boris Zbarsky [:bz] 2012-07-19 21:42:02 PDT
Peter, how do you think we should expose these?  We could probably do just a sequence (so nsTArray) of whatever type...
Comment 2 Boris Zbarsky [:bz] 2012-07-19 21:42:36 PDT
Created attachment 644175 [details] [diff] [review]
Fail codegen for variadic arguments until we add support for them.
Comment 3 Boris Zbarsky [:bz] 2012-07-20 00:56:33 PDT
Created attachment 644215 [details] [diff] [review]
Fail codegen for variadic arguments until we add support for them.
Comment 5 Ed Morley [:emorley] 2012-07-27 08:15:02 PDT
https://hg.mozilla.org/mozilla-central/rev/69ad0bb0bc25

Note You need to log in before you can comment on or make changes to this bug.