Building nsViewManager.cpp on Linux gives the warning: ../../../mozilla/view/src/nsViewManager.h:148: warning: `class nsViewManager' only defines a private destructor and has no friends

it uses Release() to call delete this, so the destructor does not need to be public. i think the warning is bogus in this case.

made protected to eliminate warning.

akkana, please mark this VERIFIED-FIXED when you get chance...

Yikes! Yes, that warning is gone, but now there are a bunch of new warnings in that file: nsViewManager.cpp: In method `unsigned int nsViewManager::UpdateView(class nsIView *, const struct nsRect &, unsigned int)': nsViewManager.cpp:743: warning: unsigned int format, pointer arg (arg 2) nsViewManager.cpp: In method `void nsViewManager::AddRectToDirtyRegion(class nsIView *, const struct nsRect &) const': nsViewManager.cpp:1545: warning: unused variable `unsigned int rv' nsViewManager.cpp: In method `unsigned int nsViewManager::Display(class nsIView *)': nsViewManager.cpp:1618: warning: unused variable `void * ds' nsViewManager.cpp:1616: warning: unused variable `struct nsRect wrect' I'll go ahead and mark the bug resolved since none of these warnings are really that important (the only one that looks bad is the first one, and that's in a printf so who cares?)