I will do better job in commit messages in the future.
On Sat, Feb 8, 2014 at 5:06 PM, Dirk Hohndel <[email protected]> wrote: > 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 > > > -- *Boris Barbulovski* http://mkfusion.bokicsoft.com/
_______________________________________________ subsurface mailing list [email protected] http://lists.hohndel.org/cgi-bin/mailman/listinfo/subsurface
