Hi Joachim, On Wed, 2010-12-08 at 19:34 +0100, Joachim wrote: > The attached patch implements in the ImageList class a fallback to load > a default icon if no one is found.
Wonderful - patch looks lovely I removed the extra braces for single line statements (seems right from a context or two), otherwise wonderful stuff. Should help shrink our install size yet further. > I have performed the following tests: .. > 2nd test is removing from images.zip all the icons except the default one... > Find attached a screenshot of the result. Really great testing too. Ace - I guess the way is cleared for removing all the missing icon duplicates (if they are still there). Do you think you could move the task from the Easy Hacks page into the 'completed easy hacks' page to avoid others spending time on it :-) Good stuff ! what is your next mission ? All the best, Michael. -- michael.me...@novell.com <><, Pseudo Engineer, itinerant idiot _______________________________________________ LibreOffice mailing list LibreOffice@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/libreoffice