[PATCH 3/4] MainWindow: maintain one window instance of the yearly statistics

Dirk Hohndel dirk at hohndel.org
Wed Mar 12 10:13:57 PDT 2014


On Wed, 2014-03-12 at 19:09 +0200, Lubomir I. Ivanov wrote:
> On 11 March 2014 20:38, Lubomir I. Ivanov <neolit123 at gmail.com> wrote:
> > On 11 March 2014 20:25, Dirk Hohndel <dirk at hohndel.org> wrote:
> >> On Tue, 2014-03-11 at 18:31 +0200, Lubomir I. Ivanov wrote:
> >>> From: "Lubomir I. Ivanov" <neolit123 at gmail.com>
> >>>
> >>> This is a bit tricky because we are using a plain widget for
> >>> a window and don't have a class for it (req. more source files).
> >>
> >> We already have 140 source files. If it takes a few more to architect
> >> things better, so be it.
> >> Also it's not a hard requirement for every class to have its own
> >> source / header files (see models.cpp/.h)
> >>
> >
> > one issue is that there won't be much in the eventual YearlyStatistics class.
> > just a constructor and destructor.
> >
> > perhaps if we add more things later in there the new class can be considered.
> >
> 
> decision on this one?
> do we need a separate class for 5-10 lines of code?

NO, I don't think we do.

/D



More information about the subsurface mailing list