SubsurfaceAbout class cleanup.
Dirk Hohndel
dirk at hohndel.org
Sat Feb 8 08:06:39 UTC 2014
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
More information about the subsurface
mailing list