← Back to team overview

kicad-developers team mailing list archive

Re: Using OPT types

 

On 11/24/19 7:12 AM, Jeff Young wrote:
> Personally I hate OPT (because it’s somewhat harder to read and more-than-somewhat harder to debug).

I tend to agree with Jeff.  The older I get, the less I like having to
dig around the code base to figure out what is going on because
templated code does tend to be less fun to debug.
> 
> I also dislike auto, except in the case of stl::’s overly-verbose iterators.  Again, they make the code harder to read more often than not.

I'm seriously rethinking typedefs as well.  I can never seem to remember
the type they represent so I have to dig around the source to figure out
the actual type.  I'm thinking just typing out the full type is easier
in the shortened typedefed version which was most likely only created to
save some typing.

> 
> Maybe I’m just showing my age….

Funny how age changes your perspective.

> 
>> On 24 Nov 2019, at 11:13, Ian McInerney <Ian.S.McInerney@xxxxxxxx> wrote:
>>
>> What is the current consensus on using OPT types in the code? I know there are some instances where we are already using them from the Boost library (since our C++ version isn't high enough to include them), but is that considered a good type to use more of?
>>
>> I am curious, because I am thinking of replumbing the position storage in the tool events to use OPTs for the position, because that will allow for cleaner handling of the position in the tools, and also because I need to pass the positions into the selection routines, and being able to pass an OPT will greatly simplify things (I think).
>>
>> -Ian
>> _______________________________________________
>> 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