On 3/2/26 9:50 PM, Omar Polo wrote:
> looks fine, but upon a closer look I realized that the similar check in
> buffsave() is now redundant.  there are only two callers:
> 
>  - filesave() which we are modifying, and
> 
>  - anycb() (in buffer.c) which attempts to write all buffers, but it
>    already has the (*bp->b_fname) != 0 and the BFCHG flag check to guard
>    it.
> 
> so, while we're here, I'd propose to also remove the BFCHG check from
> buffsave(), effectively "bubbling it up" to filesave().
> 
> what do you think?

Wonderful! You hit the nail on the head. It's a big improvement indeed.

BR
Han