[Geany-devel] multiple instance save-settings - Re: Race condition when saving geany.conf

[Lex Trotman]

2010/6/3 Enrico Tröger <enrico.troeger at uvena.de>:
> On Sun, 30 May 2010 22:43:54 +0400, Eugene wrote:
>
>> > > > > Or, as another option, we can remove automatic saving of
>> > > > > preferences when closing. As preferences are saved immediately
>> > > > > after Preferences dialog is closed, this option wouldn't do
>> > > > > any harm. Or would it?
>> > > >
>> > > > The historical reason for save on exit was to store window size
>> > > > and position and maybe other things (scribble content, maybe
>> > > > more). So, we still need to save on exit but maybe we can lower
>> > > > the amount of settings we want to save on exit, only those which
>> > > > are really necessary.
>> > > >
>> > >
>> > > (this is also a reply to Dimitar)
>> > >
>> > > Well, then it's not a big deal to only save_dialog_prefs() after
>> > > the Preferences dialog is closed and save everything but dialog
>> > > prefs (UI prefs, session etc.) on exit. That is logical and will
>> > > solve the problem.
>> >
>> > Sounds ok to me. But I didn't have a look at the code to find maybe
>> > some potential problems with such a change.
>> >
>>
>> Okay.  Just for clarity, the discussion about this topic has moved to
>> a new thread <http://lists.uvena.de/geany-devel/2010-May/002358.html>.
>> Dimitar and Lex have already posted there some patches.
>
> Oops, sorry.
> I still didn't manage to follow the thread. It's currently way too
> time-consuming and also, as you know, I'm not such a big fan of this
> multiple instance stuff anyway. Geany was never meant to work like this.

Yep, and I wasn't interested in multi instance either until I realised
that it allows me to have Geany on multiple screens, I do  lot of
switching back and forth between .hpp and .cpp files so showing them
both would be a *big* help.  Helping Eugene and Ditmar get a few
wrinkles out of the multiple instance issues (which are mostly exposed
by the session management stuff) is easier, no way easier, than making
Geany do multiple windows.

> Anyway, I'll concentrate on the upcoming release for now and read and
> reply in the mentioned thread later (hopefully still this weekend).

Correct prioritisation, 0.19 is safe enough (decided after much
discussion :), although there are some patches which should go on
0.20, or merge one of the sm implementations each of which contains
them.

Cheers
Lex
>
>
> Regards,
> Enrico
>
> --
> Get my GPG key from http://www.uvena.de/pub.asc
>
> _______________________________________________
> Geany-devel mailing list
> Geany-devel at uvena.de
> http://lists.uvena.de/cgi-bin/mailman/listinfo/geany-devel
>
>