bug#54156: 28.0.91; set-face-attribute with a nil FRAME doesn't change the default

[Lars Ingebrigtsen]

Damien Cassou <damien@cassou.me> writes:

> I have this in my init.el file:
>
> (set-face-attribute 'finsit-javascript-html-tag-face nil :background nil)

It's perhaps easier to reproduce with a standard face:

(set-face-attribute 'region nil :background nil)

Eval this and mark some area and see that it marking a region no longer
has a background.  `C-x 5 2' and everything is back to normal in the new
frame.

However, this works:

(set-face-attribute 'region nil :background "red")

So it looks like setting something to nil is forgotten?  I.e., there's
no difference between nil and "use the default", I'm guessing, but I
haven't tried to debug further.  Anybody familiar with the code in this
area?

-- 
(domestic pets only, the antidote for overdose, milk.)
   bloggy blog: http://lars.ingebrigtsen.no