[Geany-devel] [PATCH 19/19] Use per document indent sizes

Jiří Techet techet at xxxxx
Mon Aug 9 21:45:02 UTC 2010

On Sun, Aug 8, 2010 at 04:40, Erik de Castro Lopo
<mle+tools at mega-nerd.com> wrote:
> Erik de Castro Lopo wrote:
>> Jiri,
>> I've applied this patch to my own tree and I'm not sure its working
>> as it should.
>> When I have a document open (with default tabs width set to 4) and I use
>> your menu item to set the document to 8 I expect the indent to switch
>> from 4 to 8 but that doesn't happen.
>> Did my cherry picking of this patch miss out something that was in one
>> of the other patches?
> Hmm, for some reason your code expected "_8" while on my machine I
> was getting "4". The attached patch handles both variants.


I wanted to avoid 8 callback functions for each menu entry by reading
the label but apparently it wasn't a good idea. When thinking about
it, the label can be localized and completely changed so I shouldn't
rely on its value.

Instead of using your patch I think it's cleaner to really use the 8
callbacks which should fix all the problems. So please use the
attached patch instead of the original one.



> Cheers,
> Erik
> --
> ----------------------------------------------------------------------
> Erik de Castro Lopo
> http://www.mega-nerd.com/
> _______________________________________________
> Geany-devel mailing list
> Geany-devel at uvena.de
> http://lists.uvena.de/cgi-bin/mailman/listinfo/geany-devel
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0019-Use-per-document-indent-sizes.patch
Type: text/x-patch
Size: 61794 bytes
Desc: not available
URL: <http://lists.geany.org/pipermail/devel/attachments/20100809/435439c4/attachment.bin>

More information about the Devel mailing list