On Thu, Jun 09, 2016 at 03:01:51AM +0100, Richard Russon wrote:
> 
> > +        snprintf (fmt, sizeof (fmt), "%%%sc", prefix);
> > +        snprintf (dest, destlen, fmt, b->new ? 'N' : ' ');
> 
> I'd have gone for a string, myself, as it's slightly more flexible.
> (The new sidebar_format wouldn't need to change)
> 
>     snprintf (fmt, sizeof (fmt), "%%%ss", prefix);
>     snprintf (dest, destlen, fmt, b->new ? "N" : "");

I don't care too strongly, but I was following the convention of %N in
$folder_format and %S, %T, and %Z in $index_format.

If more flexibilty it needed, you can also use a conditional construct:
  %?n?N?

-- 
Kevin J. McCarthy
GPG Fingerprint: 8975 A9B3 3AA3 7910 385C  5308 ADEF 7684 8031 6BDA

Attachment: signature.asc
Description: PGP signature

Reply via email to