> From: Ship Mints <shipmints@gmail.com>
> Date: Wed, 30 Apr 2025 09:10:05 -0400
> Cc: Eli Zaretskii <eliz@gnu.org>, 77715@debbugs.gnu.org, drew.adams@oracle.com
>
> Face flashing isn't a feature of bell ringing, it's the opposite.
??? Then why is ring-bell-functions being mentioned?
> I'd consider putting the face flashing code with
> face-related code so it's clear faces can be flashed anywhere for any reason the users want.
But this code is not general enough for that. It was written
explicitly for additional optional values for ring-bell-functions, as
the doc strings say.
The prototype was derived from private code used in this narrow case but if we're going to adopt the functionality in core, I think we go the extra mile to make it appropriately general.