[PATCH] Save more planner settings into prefs

2015-03-24 Thread Gaetan Bisson
000 Subject: [PATCH] Save more planner settings into prefs This adds to the prefs struct the variables last_stop, verbatim_plan, display_runtime, display_duration, and display_transitions from the planner so their values are saved from one session to the next. Signed-off-by: Gaetan Bisson --- pre

Re: [PATCH] Save more planner settings into prefs

2015-03-24 Thread Lubomir I. Ivanov
On 25 March 2015 at 00:19, Gaetan Bisson wrote: > Hi, > > The attached patch adds more variables from the planner to the prefs > struct so that their values are saved from one session to the next. > hello Gaetan, any reason to also update the plannerSettings.ui file; perhaps residue from tests w

Re: [PATCH] Save more planner settings into prefs

2015-03-24 Thread Gaetan Bisson
Hi Lubomir, [2015-03-25 00:27:20 +0200] Lubomir I. Ivanov: > any reason to also update the plannerSettings.ui file; perhaps residue > from tests while you were developing the patch? > the commit notes don't explain this particular change. The new code in subsurfacestartup.c takes care of initiali

Re: [PATCH] Save more planner settings into prefs

2015-03-24 Thread Gaetan Bisson
[2015-03-24 14:30:57 -1000] Gaetan Bisson: > The new code in subsurfacestartup.c takes care of initializing those > settings, so there is no need to set default values for the widgets in > plannerSettings.ui . Note that only two of those five widgets were given explicit default values in plannerSe

Re: [PATCH] Save more planner settings into prefs

2015-03-25 Thread Lubomir I. Ivanov
On 25 March 2015 at 02:34, Gaetan Bisson wrote: > [2015-03-24 14:30:57 -1000] Gaetan Bisson: >> The new code in subsurfacestartup.c takes care of initializing those >> settings, so there is no need to set default values for the widgets in >> plannerSettings.ui . > > Note that only two of those fiv

Re: [PATCH] Save more planner settings into prefs

2015-03-25 Thread Gaetan Bisson
tter commit message. -- Gaetan >From 835e6efdbef6607a41c20f242fdf5b689fb97778 Mon Sep 17 00:00:00 2001 From: Gaetan Bisson Date: Tue, 24 Mar 2015 11:57:22 -1000 Subject: [PATCH] Save more planner settings into prefs This adds to the prefs struct the variables last_stop, verbatim_plan, display_run