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