kicad-developers team mailing list archive
-
kicad-developers team
-
Mailing list archive
-
Message #19014
Re: APP_SINGLE_TOP in 5834
I didn't see why either but I also did not test it to confirm. If you
have the time to confirm, I would appreciate it.
On 6/28/2015 11:48 PM, Chris Pavlina wrote:
> Interesting. I don't see why - perhaps I'll have a look at that tomorrow.
>
> On Sun, Jun 28, 2015 at 07:27:40PM -0400, Wayne Stambaugh wrote:
>> The author of this code informed me that there may be some issues if an
>> exception occurs during APP_SINGLE_TOP initialization. I did not
>> confirm this but I suspect that he may be correct. That is why I
>> conditionally compiled them for the time being until it can be confirmed.
>>
>> On 6/28/2015 4:22 PM, Chris Pavlina wrote:
>>> Hi,
>>>
>>> Just curious here - what was the reason behind the change made in
>>> single_top.cpp in revision 5834?
>>>
>>> http://bazaar.launchpad.net/~kicad-product-committers/kicad/product/revision/5834
>>>
>>> I had moved Pgm().OnPgmExit() into OnExit() to correct a segfault on
>>> quit - why move it back to OnRun() for non-Python builds? Surely OnExit
>>> is the right place for exit code in all cases?
>>>
>>> --
>>> Chris
>>>
>>> _______________________________________________
>>> 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