On Sun, 18 Sep 2011 18:24:53 +0200, David Goldwich <[email protected]> wrote: > > >> + if (ch) > >> + avio_w8(dynbuf, 0); > >> + > >> + len = avio_close_dyn_buf(dynbuf, (uint8_t **)dst); > > > > Why not make dst uint8_t** and avoid the cast? > > Hm, if you don't want to cast in another place than, you have to make > all the other strings uint8_t*, e.g. in the ID3v2ExtraMetaGEOB struct. > Is that ok? I think it hides away a bit the intention of being a string, > but if it's fine for you it's fine for me. >
uint8_t* is routinely used for utf-8 strings all over libav, so i see no problem with it. -- Anton Khirnov _______________________________________________ libav-devel mailing list [email protected] https://lists.libav.org/mailman/listinfo/libav-devel
