[FFmpeg-devel] [PATCH v4 1/4] avutil/{avstring, bprint}: add XML escaping from ffprobe to avutil

Nicolas George george at nsup.org
Fri Feb 19 15:24:04 EET 2021


Anton Khirnov (12021-02-19):
> Why not have a single ESCAPE_MODE_XML mode and use flags to tweak what
> specifically is or is not escaped?

I forgot the flags argument were there. You are right, it is the best
choice: escape <&> by default, a flag to escape ' and a flag to escape
".

> Mild pereference for "attr" here
> Besides, doesn't everyone use auto-completion for identifiers?

Thanks you for your input. Typing the identifiers is not the only issue:
overly long lines are annoying to read the code too.

Regards,

-- 
  Nicolas George
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 833 bytes
Desc: not available
URL: <https://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20210219/1540bb73/attachment.sig>


More information about the ffmpeg-devel mailing list