← Back to team overview

kicad-developers team mailing list archive

Re: [PATCH] common.h tidyups

 

Hi Wayne,

Yes, first in the list is "8f192fe12 OPENGL_GAL: Init currentTarget"

That particular one is actually an older patch I had on my tidy-up
branch and isn't really in the common.h set, but it keeps my IDE
quieter! 5e6a022b5 is the first common.h one.

Cheers,

John

On Thu, Feb 9, 2017 at 9:56 PM, Wayne Stambaugh <stambaughw@xxxxxxxxx> wrote:
> John,
>
> I like this idea.  Anything that prevents unnecessary compiling is a
> good thing.  Looking at this repo, I'm guessing that everything from
> commit 8f192fe12eb52efd6d024c82d1416db5e62b345f on is what needs to be
> merged.  I'll try to take a look at it over the weekend.  Keep up the
> great work.
>
> Thanks,
>
> Wayne
>
> On 2/9/2017 8:44 AM, John Beard wrote:
>> Hi,
>>
>> I have a big set of patches on a branch that are mostly to do with
>> stripping seldom-used functions out of common.h into more targeted
>> headers:
>>
>> https://code.launchpad.net/~john-j-beard/kicad/+git/kicad/+ref/tidy_ups
>>
>> The idea here is to reduce the visibility of these functions from
>> almost every file in KiCad to just those that need them. This also
>> means that changing the signatures (or even just adding comments)
>> doesn't force a huge recompile of the whole codebase, just the files
>> that use them. It also makes common.h less of a "dumping ground" for
>> random stuff.
>>
>> There were several old functions or variables that weren't used,
>> weren't defined, or weren't declared too, and these were tidied up.
>>
>> I have used std::unique_ptr in a couple of places, at one was leaking
>> an object before.
>>
>> I only meant to do a few, but since any big work in common.h is as bad
>> as small work, I did quite a few.
>>
>> The patches should mostly be cherry-pickable in case not all of them are wanted.
>>
>> Cheers,
>>
>> John
>>
>> _______________________________________________
>> Mailing list: https://launchpad.net/~kicad-developers
>> Post to     : kicad-developers@xxxxxxxxxxxxxxxxxxx
>> Unsubscribe : https://launchpad.net/~kicad-developers
>> More help   : https://help.launchpad.net/ListHelp
>>
>
> _______________________________________________
> Mailing list: https://launchpad.net/~kicad-developers
> Post to     : kicad-developers@xxxxxxxxxxxxxxxxxxx
> Unsubscribe : https://launchpad.net/~kicad-developers
> More help   : https://help.launchpad.net/ListHelp


Follow ups

References