[Geany-devel] allow always save all fix

[Enrico Tröger]

On Fri, 20 May 2011 20:30:45 +0300, Dimitar wrote:

>Hi,
>
>With allow_always_save=true, both Save and Save All are always enabled
>(unless there are no files, of course). Obviously, that makes sense for
>Save. But what you invoke Save All and there are no modified files? It
>doesn't save all files regardless of modification, as you may think,
>nor does it save the current file - in fact, it does nothing, and
>allow_always_save has no effect on it. So IMHO, en/disabling of Save
>All should not be affected by the value of allow_always_save.

Agreed.
Just for your interest, the original request for this feature is here:
http://www.mail-archive.com/geany@uvena.de/msg01148.html


>The attached patch does that, and makes the preference changeable,

I committed the patch, thanks.


>provided that you call ui_save_buttons_toggle() after change (we'll
>need this when/if the various preferences editor is included in Geany).

Not completely sure what you mean here.

Regards,
Enrico

-- 
Get my GPG key from http://www.uvena.de/pub.asc
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: not available
URL: <http://lists.geany.org/pipermail/devel/attachments/20110522/89db4d6c/attachment.pgp>