[MPlayer-dev-eng] [PATCH 2/4] String handling audit/cleanup

[Ivan Kalvachev]

I haven't looked in the code specifics.
However I'll make one proposal I had in mind for quite long time.

Why instead of sprintf/snprints we don't use asprintf() - the function
stores the result in newly allocated buffer.  It is GNU extension, but
as Rich have already pointed on few occasions it is quite easy to be
recreated with using snprintf() and realloc.