On Sat, 2014-02-08 at 08:50 +0100, Boris Barbulovski wrote: > > > An SubsurfaceAbout class cleanup. > > > Main change is that now SubsurfaceAbout doesn't have instance object, > but it constructs and destructs dynamically.
Based on Tomaz' ACK I have taken the patch... > If small patch it's acceptable, and continue with this pattern by > remove other instance() members(around 10). Yes, small patches like this are great - this is basically my preferred granularity. But please pay attention to your commit messages - that was mixed up in this patch. Take a look at what I pushed to see what I mean. /D _______________________________________________ subsurface mailing list [email protected] http://lists.hohndel.org/cgi-bin/mailman/listinfo/subsurface
