bug#32404: Format fails to copy text properties from the format-string for adjacent %-sequences

classic Classic list List threaded Threaded
4 messages Options
Reply | Threaded
Open this post in threaded view
|

bug#32404: Format fails to copy text properties from the format-string for adjacent %-sequences

Andrea Cardaci
Basically what the title says, take a look at this example:

(format (concat (propertize "%s" 'face 'bold)
                "" ; any non-empty string works
                (propertize "%s" 'face 'error))
        "foo" "bar")
#("foobar" 0 6 (face bold))

This has also been described at https://emacs.stackexchange.com/q/43960/13744


Andrea



Reply | Threaded
Open this post in threaded view
|

bug#32404: Format fails to copy text properties from the format-string for adjacent %-sequences

Eli Zaretskii
> From: Andrea Cardaci <[hidden email]>
> Date: Thu, 9 Aug 2018 02:30:55 +0200
>
> Basically what the title says, take a look at this example:
>
> (format (concat (propertize "%s" 'face 'bold)
>                 "" ; any non-empty string works
>                 (propertize "%s" 'face 'error))
>         "foo" "bar")
> #("foobar" 0 6 (face bold))
>
> This has also been described at https://emacs.stackexchange.com/q/43960/13744

Thanks, I believe I fixed this on the emacs-26 branch.



Reply | Threaded
Open this post in threaded view
|

bug#32404: Format fails to copy text properties from the format-string for adjacent %-sequences

Andrea Cardaci
> Thanks, I believe I fixed this on the emacs-26 branch.

Yes, I just tried and it seems to work. Thank you so much for the
prompt response and fix.


Andrea



Reply | Threaded
Open this post in threaded view
|

bug#32404: Format fails to copy text properties from the format-string for adjacent %-sequences

Eli Zaretskii
> From: Andrea Cardaci <[hidden email]>
> Date: Thu, 9 Aug 2018 22:25:26 +0200
> Cc: [hidden email]
>
> > Thanks, I believe I fixed this on the emacs-26 branch.
>
> Yes, I just tried and it seems to work. Thank you so much for the
> prompt response and fix.

Thanks for verifying, I'm therefore closing this bug.