← Back to team overview

kicad-developers team mailing list archive

Re: [PATCH] Fix lp:1529159

 

On 2/22/2016 4:07 PM, Clemens Koller wrote:
> Hello, Wayne!
> 
> On 2016-02-22 20:23, Wayne Stambaugh wrote:
>> Anyone else see any issues with this patch?  I just tested it on windows
>> and I didn't see any issues.  Since this only seems to be an issue on
>> OSX, maybe we should wrap this in an #ifdef/#endif as much as I don't
>> like them.
> 
> I don't see a reason to wrap that. Don't draw a zero sized object seems
> to be fair. I would even consider to emit a warning in debug mode when this
> triggers...

I'm not.  I will add a note about the wxWidgets bug so whenever we set
the minimum wxWidgets version to 3.0.3 or later, it can be removed.

> 
> Regards,
> 
> Clemens
> 
>>
>> On 2/20/2016 8:13 AM, Bernhard Stegmaier wrote:
>>> Hi,
>>>
>>> attached a patch to fix lp:1529159, it is still reproducible with a current wxWidgets master.
>>> Credits go to the author of the bug report, I just did change the condition to immediately return if one of the sizes is zero (and create the patch).
>>>
>>> Of course, it could be argued if the real fix for this should go into OS X wxWidgets (I guess it should), but not trying to draw something zero sized at all doesn’t seem like a bad thing in general.
>>>
>>>
>>> Regards,
>>> Bernhard
>>>
>>>
>>>
>>>
>>>
>>>
>>>
>>> _______________________________________________
>>> 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
>>
> 
> _______________________________________________
> 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