Closed Bug 506201 Opened 10 years ago Closed 10 years ago
.cpp fails to compile on opensolaris: goto bypasses variable init
http://hg.mozilla.org/mozilla-central/rev/b837948c1daf#l1.1139 1.1140 goto out; 1.1141 } 1.1142 rv = rc.compile(cx); 1.1143 + static int fail = 0; // TODO: remove 1.1144 + if (!rv) 1.1145 + ++fail; 1.1146 + debug_only_printf(LC_TMRegexp, "## Fail? %d, Total %d\n", (int)!rv, fail); 1.1147 out: That's the only error when compiling. But I suppose it's TODO remove anyway ;)
I suppose until it actually gets removed, putting that added code in its own block should fix it.
Comment on attachment 390427 [details] [diff] [review] patch I think we should just remove that code. Its clearly not meant to be there.
Yep, totally personal debugging comment that should have been removed. Sorry about that! (I can't commit it, but the patch is right.)
Assignee: general → leon.sha
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.