bug-gnu-emacs
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

bug#10736: Add "underwave" face attribute


From: Jan Djärv
Subject: bug#10736: Add "underwave" face attribute
Date: Fri, 13 Apr 2012 19:49:24 +0200

Hello.

12 apr 2012 kl. 23:57 skrev Stefan Monnier:

>> Looking at the mailing list, it seems like it was agreed that this is
>> useful, and that this interface is correct.  Since we're in Emacs 24.2
>> territory now, should this go in?
> 
> Assuming the code is OK, yes.   I'd like to hear someone's opinion on
> the code, tho (Jan, maybe?).
> 

Looks OK.  In general I don't like the "Get GC values, change some GC value, 
draw, restore changed GC value" technique. It is better to create more GC:s.  
But the new code just mimics the old, so it is nothing particular for this 
patch.

        Jan D.






reply via email to

[Prev in Thread] Current Thread [Next in Thread]