eeschema: Bug and assertion failure when changing plot output directory
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
KiCad |
Fix Released
|
Low
|
Unassigned |
Bug Description
There are three issues mentioned here. They are somewhat interrelated, so I will list them here in one bug:
1) Upon closing the 'Plot Schematic' dialog after changing the output directory and plotting the schematics, a full path is passed to wxFileDialog, causing the aforementioned assert (assert causing no adverse effects)
2) The output directory information is not saved to the project config file unless a plot command is issued (bug)
3) IMO I don't think the project file dialog should be displayed at all upon closing the plot schematic dialog (personal preference)
I've attached a patch which corrects the aforementioned issues.
ASSERT INFO:
../src/
BACKTRACE:
[1] wxFileName:
[2] wxFileDialog:
[3] wxFileDialog:
[4] wxAppConsoleBas
[5] wxEvtHandler:
[6] wxEventHashTabl
[7] wxEvtHandler:
[8] wxEvtHandler:
[9] wxEvtHandler:
[10] wxWindowBase:
[11] wxEvtHandler:
[12] wxMenuBase:
[13] g_closure_invoke
[14] g_signal_
[15] g_signal_emit
[16] g_main_
[17] g_main_loop_run
[18] gtk_main
[19] wxGUIEventLoop:
[20] wxEventLoopBase
[21] wxAppConsoleBas
[22] APP_KICAD::OnRun() /home/user/
[23] wxEntry(int&, wchar_t**)
[24] main /home/user/
[25] __libc_start_main
[26] _start
Application: kicad
Version: (2015-07-24 BZR 5989, Git 9b9c794)-product debug build
wxWidgets: Version 3.0.2 (debug,
Platform: Linux 3.19.0-23-generic x86_64, 64 bit, Little endian, wxGTK
Boost version: 1.54.0
Changed in kicad: | |
importance: | Undecided → Low |
Changed in kicad: | |
status: | Fix Committed → Fix Released |
I had forgotten that the output directory can be changed without selecting 'Browse'. Therefore, it makes sense to pull that logic into a OnText event handler instead. I've updated the patch accordingly.