interface IDBObjectStoreSync { boolean delete (any key); } The code generator produces a method for this in IDBObjectStoreSyncBinding.cpp which doesn't compile since it conflicts with the C++ reserved word "delete" Attaching a quick fix ...

Same problem with c++ reserved word "continue". interface IDBCursorSync { boolean continue(optional any key); boolean delete(); }; New version of quick fix added.

I would prefer a method that takes the WebIDL identifier name and produces the name used in the binding code. Then we can just call this method from the various places it's needed and update it as it becomes necessary.... Jan, Vendelin, are you actively working on this?

(In reply to Boris Zbarsky (:bz) from comment #2) > I would prefer a method that takes the WebIDL identifier name and produces > the name used in the binding code. Then we can just call this method from > the various places it's needed and update it as it becomes necessary.... > > Jan, Vendelin, are you actively working on this? Yes, he needs this for bug 798875

I have created new attachment with fix and simple test. Any suggestions are welcome.

Comment on attachment 676388 [details] [diff] [review] fix for c++ keywords v.3 This doesn't seem to be a patch against tip, but against some intermediate state. I'd really like to see a diff against tip. Also, please drop the 'CG' from the class name, since this is not in fact a subclass of CGThing. And add the missing newline to the end of TestCodeGen.webidl, please. r=me with that.

New version of patch against tip added in attachment.

Comment on attachment 676990 [details] [diff] [review] bug-806184-patch r=me