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

[Lubomir I. Ivanov]

On 12 March 2014 19:13, Dirk Hohndel <dirk at hohndel.org> wrote:
> 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.
>

bump. same patch attached just in case.
it's basically the same as the original in this thread [PATCH 3/4].

lubomir
--
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-MainWindow-maintain-one-window-instance-of-the-yearl.patch
Type: application/octet-stream
Size: 3776 bytes
Desc: not available
URL: <http://lists.hohndel.org/pipermail/subsurface/attachments/20140313/ef9b4ee1/attachment.obj>