symbol editor: Assert on "enter" key

Also in eeschema the same behaviour, with the same message.
--------------------------------------

ASSERT INFO:
/home/dinoghi/SANDBOXKICAD5/src/kicad/include/tool/tool_event.h(448): assert "HasPosition()" failed in returnCheckedPosition(): Attempted to get position from non-position event

BACKTRACE:
[1] make_fcontext

Odd that I don't get that assert on OSX.

In any case it's harmless (if annoying). You can "Continue" without harm.

Now I can see it in pcbnew also.
gerbv seems to work without this issue.

Update: In eeschema if any tool is selected the issue does not appear.

In pcbnew if I end a "circle" command with enter I get a segmentation fault.
Also if I end a "add graphic line" using enter.
Also "add text on copper layer or graphic text"

"Add graphic polygon" works fine.

Using left click works fine. May be this is related to the enter key behaviour I'm experiencing, so I don't know if I should file a new bug for this.

Ending the "add graphic circle" with enter gdb gives:

Thread 1 "pcbnew" received signal SIGSEGV, Segmentation fault.
0x00007fffe08b8a79 in jump_fcontext () from /home/dinoghi/SANDBOXKICAD5/usr/bin/_pcbnew.kiface
(gdb) cont
Continuing.

Program terminated with signal SIGSEGV, Segmentation fault.
The program no longer exists.
(gdb)

I don't get the segfault ending a circle with an <enter>, but it does start a new circle at the same point, which isn't right.

Let me fix that and see if your segfault also goes away. If, not, we should put it under a separate bug.

Fixed in revision ef5c69167a493683c81d44cf752fc141dcf174de
https://git.launchpad.net/kicad/patch/?id=ef5c69167a493683c81d44cf752fc141dcf174de

On 13/05/19 23:25, KiCad Janitor wrote:
> Fixed in revision ef5c69167a493683c81d44cf752fc141dcf174de
> https://git.launchpad.net/kicad/patch/?id=ef5c69167a493683c81d44cf752fc141dcf174de
>
> ** Changed in: kicad
> Status: In Progress => Fix Committed
>

Testing commit ef5c69167a493683c81d44cf752fc141dcf174de: works for me too.

Also the segfault on pcbnew when ending drawing operations with enter is
working for me now.

Now I have a new assert launching pcbnew "Assert failure" failed in
UpdateHotKeys(): Duplicate hotkey definitions for <unknown>:
pcbnew.Control.switchUnits and common.Control.toggleUnits", but I think
this is a new bug, not related to the "enter" assert.

Cheers,
Dino.

On 14/05/19 09:56, Dino Ghilardi wrote:
> On 13/05/19 23:25, KiCad Janitor wrote:
>> Fixed in revision ef5c69167a493683c81d44cf752fc141dcf174de
>> https://git.launchpad.net/kicad/patch/?id=ef5c69167a493683c81d44cf752fc141dcf174de
>>
>>
>> ** Changed in: kicad
>>         Status: In Progress => Fix Committed
>>
>
> Testing commit ef5c69167a493683c81d44cf752fc141dcf174de: works for me too.
>
> Also the segfault on pcbnew when ending drawing operations with enter is
> working for me now.
>
> Now I have a new assert launching pcbnew "Assert failure" failed in
> UpdateHotKeys(): Duplicate hotkey definitions for <unknown>:
> pcbnew.Control.switchUnits and common.Control.toggleUnits", but I think
> this is a new bug, not related to the "enter" assert.
>
> Cheers,
> Dino.

Also the key is used in pcbnew/tools/pcbnew_control.cpp

TOOL_ACTION PCB_ACTIONS::switchUnits( "pcbnew.Control.switchUnits",
         AS_GLOBAL, TOOL_ACTION::LegacyHotKey( HK_SWITCH_UNITS ),
         "", "" );

and common/tool/actions.cpp
TOOL_ACTION ACTIONS::toggleUnits( "common.Control.toggleUnits",
+ AS_GLOBAL, TOOL_ACTION::LegacyHotKey( HK_SWITCH_UNITS ),
+ _( "Switch units" ), _( "Switch between inches and millimeters"
...

so it is not possible to work it around changing the hotkey assignment.

Cheers,
Dino.

Yeah, the PCB_ACTION is no longer needed now that there's a COMMON equivalent. I'll remove the PCBNew one (and GerbView has a duplicate as well).