>> This has such regression that before it supported the following workflow: >> in the minibuffer 'M-n M-n ...' used to pull hi-lock faces from the ordered >> "future history" list. But read-face-name doesn't support this feature. >> >> If highlight-regexp should use read-face-name, then this feature >> could be added to read-face-name. Here is the simplest patch, >> but it would be a tough task to explain the need of an additional >> DEFAULTS arg in the documentation. An alternative would be to >> rewrite read-face-name and keep cdr of the existing arg DEFAULT >> for the future history of the minibuffer. > > Hm, I assumed that that was what read-face-name did (since it allows > DEFAULT to be a list). Then read-face-name should be improved to support a list of defaults: