nsID claims it is frozen. This is certainly correct for the struct layout, but I'm not sure whether it means we nsID::Parse and nsID::ToString, which are dllexports. If we *did* freeze them, we need to expose them from xpcom.dll and not just xpcom_core.dll. In any case, I would like to make both of these frozen C exports NS_ParseID and NS_IDToString (and add inline wrappers for existing callers).

Marking blocking 1.8 at least until we the frozen-API issues are sorted out.

No, nsID::Parse and nsID::ToString are not frozen. The exports have mangled names, so they are not in any way ready to be frozen. I agree, that we should expose those APIs.

This simply moves the impl to the glue, instead of exposing it via a frozen export which I believe is unnecesary. This also needs cvsmoves which I will attach separately.

Comment on attachment 190145 [details] [diff] [review] Move nsID symbols to the glue, rev. 1 nice! r=darin

Comment on attachment 190145 [details] [diff] [review] Move nsID symbols to the glue, rev. 1 a=shaver

Justdave/shaver, can one of you do the cvsmoves?

Reassigning to Dave for cvsmoves, please assign back to me once those are complete.

cvsmove completed, give it about 5 min for cvs-mirror to pick it up, then please verify.

Fixed on trunk for 1.8b4. Thank you Dave.