I would like to nominate image/src/* and in particular its class hierarchy which completely doesn’t make any sense what so ever. imgRequest, imgIRequest, we got it all.
Andreas On Oct 16, 2014, at 6:44 PM, Randell Jesup <rje...@jesup.org> wrote: >> On Fri, Oct 17, 2014 at 1:32 AM, Nicholas Nethercote <n.netherc...@gmail.com >>> 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... >>> >> >> Probably not the worst, but always deserves a mention: >> http://dxr.mozilla.org/mozilla-central/source/layout/xul/nsSprocketLayout.cpp#632 >> >> // That's it! If you made it this far without having a nervous >> // breakdown, congratulations! Go get yourself a beer. >> >> ... which ties into the XUL thread :-) > > Reminds me of the total rewrite I did of the table-layout code for > SpyGlass eons ago, to actually work correctly (I think I even had it > handle bug 10212 correct - height 100% when nesting tables). Quite > brain-warping, especially how changes can ripple through multiple times. > > -- > Randell Jesup, Mozilla Corp > remove "news" for personal email > _______________________________________________ > dev-platform mailing list > dev-platform@lists.mozilla.org > https://lists.mozilla.org/listinfo/dev-platform _______________________________________________ dev-platform mailing list dev-platform@lists.mozilla.org https://lists.mozilla.org/listinfo/dev-platform