On Fri, Oct 17, 2014 at 1:32 AM, Nicholas Nethercote <[email protected]> wrote: > On Thu, Oct 16, 2014 at 11:32 PM, Nicholas Nethercote > <[email protected]> wrote: >> >> I was wondering what people think is the worst piece of code in the >> entire Mozilla codebase. I'll leave the exact meanings of "worst" and >> "piece of code" unspecified... > > Thanks for the replies so far! I deliberately left this question vague > to see what kind of responses people would give. But mostly I'm > interested in code whose awfulness impacts users in a serious way.
Well, most of the time the user-facing correctness problems are not serious, but nsDocLoader::mIsLoadingDocument is very bogus, causes correctness problems and has consequences that include us still having the ghost of the old HTML parser around for about:blank. (nsDocLoader in general isn't nice code.) -- Henri Sivonen [email protected] https://hsivonen.fi/ _______________________________________________ dev-platform mailing list [email protected] https://lists.mozilla.org/listinfo/dev-platform
