Uninitialized values and memory leaks

[Dirk Hohndel]

On Wed, 2013-10-02 at 12:28 -0300, Tomaz Canabrava wrote:
> Top posting, mobile phone email, will provide a better email when I
> finish formatting my pet. 
> 
Don't let PETA hear about that...

> I prefer the "one class per file"  thing, but dirk asked me to
> separate things by "logical group", if he accepts it, I'll gladly move
> the models to a new file ( and a new folder)  

(makes retching noises)

We have 25 cpp files already. I'm not convinced I want to double that...

/D


> Em 02/10/2013 11:17, "Lubomir I. Ivanov" <neolit123 at gmail.com>
> escreveu:
>         On 2 October 2013 17:09, Lubomir I. Ivanov
>         <neolit123 at gmail.com> wrote:
>         > i'm working on that as we speak. the table under the profile
>         is simply
>         > missing at this point in the Qt build.
>         > problem is that there are some merge conflicts between this
>         patch and
>         > my local changes.
>         >
>         > my modifications affect models.*, modeldelegatates.*,
>         print*.
>         >
>         
>         actually tomaz, if you are reading this - i really think that
>         every
>         model or model delegate should be in a separate class file in
>         a folder
>         qt-ui/models, for example.
>         ATM i can only compile in a VM and every single change in
>         models.h
>         pretty much re-compiles the whole file stack and it's is quite
>         sloooow
>         over here.
>         also it can create merge conflicts as show above.
>         
>         are there structural concerns that are stopping us from doing
>         that?
>         
>         lubomir
>         --
>         _______________________________________________
>         subsurface mailing list
>         subsurface at hohndel.org
>         http://lists.hohndel.org/cgi-bin/mailman/listinfo/subsurface
> _______________________________________________
> subsurface mailing list
> subsurface at hohndel.org
> http://lists.hohndel.org/cgi-bin/mailman/listinfo/subsurface