Re: [O] [PATCH] Font-lock: allow hiding of brackets surrounding macros

2014-02-05 Thread Bastien
Hi Florian,

Bastien b...@gnu.org writes:

 Florian Beck f...@miszellen.de writes:

 I gave it a try. Well, I spend most of my time on formatting this
 patch. Hope it works.

 Works well, thanks.  Unless someone is against this change, I'll apply
 it next week.

This is now in master -- thanks!

-- 
 Bastien



Re: [O] [PATCH] Font-lock: allow hiding of brackets surrounding macros

2014-02-05 Thread Sebastien Vauban
Hello Florian and Bastien,

Bastien wrote:
 Bastien b...@gnu.org writes:
 Florian Beck f...@miszellen.de writes:

 I gave it a try. Well, I spend most of my time on formatting this
 patch. Hope it works.

 Works well, thanks.  Unless someone is against this change, I'll apply
 it next week.

 This is now in master -- thanks!

I just tested it.

I think that I would have left one pair of {} visible around macro calls
(for making it clear that it's something special, even if we can play
with the face `org-macro'); but, anyway, I find this to be a nice
improvement. Thanks for it!

Best regards,
  Seb

-- 
Sebastien Vauban




Re: [O] [PATCH] Font-lock: allow hiding of brackets surrounding macros

2014-02-05 Thread Bastien


Hi Sébastien,

Sebastien Vauban sva-news-D0wtAvR13HarG/idocf...@public.gmane.org
writes:

 I think that I would have left one pair of {} visible around macro calls
 (for making it clear that it's something special, even if we can play
 with the face `org-macro');

Playing with the face is the right thing to do I guess.

-- 
 Bastien




Re: [O] [PATCH] Font-lock: allow hiding of brackets surrounding macros

2014-01-31 Thread Bastien
Hi Florian,

Florian Beck f...@miszellen.de writes:

 I gave it a try. Well, I spend most of my time on formatting this
 patch. Hope it works.

Works well, thanks.  Unless someone is against this change, I'll apply
it next week.

-- 
 Bastien