Ideally we'd just do this when we need it, but the ostream derivation
will be controlled by both global config and per-structure config, so
it's hard to know exactly when we need it and when we don't.
Option<RenameRule> doesn't make sense anywhere because there's a
RenameRule::None itself, which is also default.
This simplifies a bit the rename rules code, and removes one weird
"default-to-gecko-case" that seems snuck by.
Using operator= is not quite sound in presence of destructors and operator
overloading.
It's perfectly fine to assume that the left-hand-side of an operator= expression
is valid memory, however we're using uninitialized memory here, that may not be
the case.
Use placement new to properly construct tagged unions. I don't need this with
any urgency, but it's the right thing to do in presence of complex types, and
the current code seems a bomb waiting to explode :)
Kartikaya Gupta
Emilio Cobos Álvarez
Jeff Muizelaar