[Webkit-unassigned] [Bug 106553] [V8] Add IDL 'enum' support to CodeGeneratorV8.pm

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=106553





--- Comment #28 from Nils Barth <nbarth at google.com>  2013-01-29 19:35:59 PST ---
(In reply to comment #24)

Thanks for replies!

(Conversion)
> Generally it's fine to use implicit conversions.  We declare many constructors "explicit", but for the ones that aren't marked "explicit", it's fine to use the implicit conversions.

Got it: explicit conversion only if required.

(const)
> Generally, we don't use "const" with String.  I don't think there's any harm in using const in this context, it's just not something we do typically.

Ok -- do we in general not use const (in V8/WebCore/Chrome), or does it vary with type or context? (Is it just "not worth bother with String"?)

I generally use const, both internally to functions and in interfaces (for usual clarity/correctness/efficiency reasons), but I understand that it can add noise, and maintaining const-correctness can be a hassle, so I was wondering what Chrome standards and best practices are -- do we use it in Chrome and if so, how widely, and anything to be aware of?

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.