Eeschema - Assertion failure when editing net labels

Bug #1168902 reported by Pekka Harjamäki
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
KiCad
Fix Released
Low
Blair Bonnett

Bug Description

Open pcbnew, click "read netlist" and voila:

ASSERT INFO:
/var/tmp/portage/x11-libs/wxGTK-2.9.4.1/work/wxPython-src-2.9.4.0/src/gtk/textctrl.cpp(846): assert "IsSingleLine()" failed in GetEditable(): shouldn't be called for multiline

BACKTRACE:
[1] wxOnAssert(char const*, int, char const*, char const*, char const*)
[2] wxTextCtrl::GetEditable() const
[3] wxTextEntry::SetMaxLength(unsigned long)
[4] DIALOG_NETLIST_FBP::DIALOG_NETLIST_FBP(wxWindow*, int, wxString const&, wxPoint const&, wxSize const&, long)
[5] DIALOG_NETLIST::DIALOG_NETLIST(PCB_EDIT_FRAME*, wxDC*, wxString const&)
[6] PCB_EDIT_FRAME::InstallNetlistFrame(wxDC*)
[7] PCB_EDIT_FRAME::Process_Special_Functions(wxCommandEvent&)
[8] wxAppConsoleBase::CallEventHandler(wxEvtHandler*, wxEventFunctor&, wxEvent&) const
[9] wxEvtHandler::ProcessEventIfMatchesId(wxEventTableEntryBase const&, wxEvtHandler*, wxEvent&)
[10] wxEventHashTable::HandleEvent(wxEvent&, wxEvtHandler*)
[11] EDA_BASE_FRAME::ProcessEvent(wxEvent&)
[12] wxEvtHandler::DoTryChain(wxEvent&)
[13] wxEvtHandler::ProcessEvent(wxEvent&)
[14] wxWindowBase::TryAfter(wxEvent&)
[15] wxAuiToolBar::OnLeftUp(wxMouseEvent&)
[16] wxAppConsoleBase::CallEventHandler(wxEvtHandler*, wxEventFunctor&, wxEvent&) const
[17] wxEvtHandler::ProcessEventIfMatchesId(wxEventTableEntryBase const&, wxEvtHandler*, wxEvent&)
[18] wxEventHashTable::HandleEvent(wxEvent&, wxEvtHandler*)
[19] wxEvtHandler::ProcessEventLocally(wxEvent&)
[20] wxEvtHandler::ProcessEvent(wxEvent&)
[21] wxEvtHandler::SafelyProcessEvent(wxEvent&)
[22] g_closure_invoke
[23] g_signal_emit_valist
[24] g_signal_emit
[25] gtk_propagate_event
[26] gtk_main_do_event
[27] g_main_context_dispatch
[28] g_main_loop_run
[29] gtk_main

Ps. pcb calculator has the same kind of bug left, see: 1167345

Tested with: kicad (BZR 4097), compiled with gcc 4.7.2 on 64-bit system. wxgtk version 2.9.4.1

Revision history for this message
Nicklas Björk (nicklas-3) wrote :

I see something similar when adding a label in eeschema in bzr 5034. Perhaps these are not related and the eeschema bug should be reported as a separate issue?

ASSERT INFO:
./src/gtk/textctrl.cpp(847): assert "IsSingleLine()" failed in GetEditable(): shouldn't be called for multiline

BACKTRACE:
[1] wxTextCtrl::GetEditable() const
[2] wxTextEntry::SetMaxLength(unsigned long)
[3] DIALOG_LABEL_EDITOR_BASE::DIALOG_LABEL_EDITOR_BASE(wxWindow*, int, wxString const&, wxPoint const&, wxSize const&, long)
[4] DIALOG_LABEL_EDITOR::DIALOG_LABEL_EDITOR(SCH_EDIT_FRAME*, SCH_TEXT*)
[5] SCH_EDIT_FRAME::EditSchematicText(SCH_TEXT*)
[6] SCH_EDIT_FRAME::CreateNewText(wxDC*, int)
[7] SCH_EDIT_FRAME::OnLeftClick(wxDC*, wxPoint const&)
[8] SCH_EDIT_FRAME::OnSelectTool(wxCommandEvent&)
[9] wxAppConsoleBase::CallEventHandler(wxEvtHandler*, wxEventFunctor&, wxEvent&) const
[10] wxEvtHandler::ProcessEventIfMatchesId(wxEventTableEntryBase const&, wxEvtHandler*, wxEvent&)
[11] wxEventHashTable::HandleEvent(wxEvent&, wxEvtHandler*)
[12] wxEvtHandler::TryHereOnly(wxEvent&)
[13] EDA_BASE_FRAME::ProcessEvent(wxEvent&)
[14] wxEvtHandler::DoTryChain(wxEvent&)
[15] wxEvtHandler::ProcessEvent(wxEvent&)
[16] SCH_EDIT_FRAME::OnHotKey(wxDC*, int, wxPoint const&, EDA_ITEM*)
[17] SCH_EDIT_FRAME::GeneralControl(wxDC*, wxPoint const&, int)
[18] EDA_DRAW_PANEL::OnKeyEvent(wxKeyEvent&)
[19] wxAppConsoleBase::CallEventHandler(wxEvtHandler*, wxEventFunctor&, wxEvent&) const
[20] wxEvtHandler::ProcessEventIfMatchesId(wxEventTableEntryBase const&, wxEvtHandler*, wxEvent&)
[21] wxEventHashTable::HandleEvent(wxEvent&, wxEvtHandler*)
[22] wxEvtHandler::TryHereOnly(wxEvent&)
[23] wxEvtHandler::ProcessEventLocally(wxEvent&)
[24] wxEvtHandler::ProcessEvent(wxEvent&)
[25] wxScrollHelperEvtHandler::ProcessEvent(wxEvent&)
[26] wxEvtHandler::SafelyProcessEvent(wxEvent&)
[27] wxWindow::GTKDoInsertTextFromIM(char const*)
[28] g_cclosure_marshal_VOID__STRINGv
[29] g_signal_emit_valist
[30] g_signal_emit_by_name
[31] g_cclosure_marshal_VOID__STRINGv
[32] g_signal_emit_valist
[33] g_signal_emit
[34] g_cclosure_marshal_VOID__STRINGv
[35] g_signal_emit_valist
[36] g_signal_emit_by_name
[37] g_closure_invoke
[38] g_signal_emit_valist
[39] g_signal_emit
[40] gtk_window_propagate_key_event
[41] g_closure_invoke
[42] g_signal_emit_valist
[43] g_signal_emit
[44] gtk_propagate_event
[45] gtk_main_do_event
[46] g_main_context_dispatch
[47] g_main_loop_run
[48] gtk_main
[49] wxGUIEventLoop::DoRun()
[50] wxEventLoopBase::Run()
[51] wxAppConsoleBase::MainLoop()
[52] APP_KICAD::OnRun() /home/nicklas/abs/kicad-bzr/src/kicad/kicad/kicad.cpp:292
[53] wxEntry(int&, wchar_t**)
[54] main /home/nicklas/abs/kicad-bzr/src/kicad/kicad/kicad.cpp:306
[55] __libc_start_main
[56] _start

summary: - PCBNew - Assertion failure
+ PCBNew - Assertion failure when reading netlist
Blair Bonnett (bcb)
summary: - PCBNew - Assertion failure when reading netlist
+ Eeschema - Assertion failure when editing net labels
Revision history for this message
Nick Østergaard (nickoe) wrote :

This bugs seems to use wxgtk 2.9. Does this mean that it is using wxwidgets 2.9? The minimum version of wx is now 3.0.0.

This bug does not report what versions are used, nor what platform it is running on. Please provide this info. Marking as incomplete.

Changed in kicad:
status: New → Incomplete
Revision history for this message
Blair Bonnett (bcb) wrote :

The assertion when reading the netlist has been fixed since this was reported, but adding or editing net labels in eeschema still causes the assertion reported by Nicklas Björk. I've posted a patch to the mailing list which fixes this -- just waiting for confirmation it doesn't break non-GTK backends and then for somebody to commit it.

Changed in kicad:
assignee: nobody → Blair Bonnett (blair-bonnett)
importance: Undecided → Low
status: Incomplete → In Progress
Revision history for this message
Blair Bonnett (bcb) wrote :

Nick -- GTK added the assertion in 2.9, so this also occurs in 3.

Revision history for this message
Blair Bonnett (bcb) wrote :

Patch committed in r5259.

Changed in kicad:
status: In Progress → Fix Committed
Revision history for this message
Nick Østergaard (nickoe) wrote :

 Blair, thank you for fixing.

Jon Neal (reportingsjr)
Changed in kicad:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.