On Wed, Nov 23, 2022 at 03:44:25PM +0100, Dmitrii Ovchinnikov wrote: > >>The types seems mismatching > > The reason for the mismatch of types is that I left the > int type according to the comments : > " > James Almer > IMO the AVFrame ones should have not been size_t to begin with, not just > because the actual dimensions you'll apply them to are int, but because > these fields are not arch dependent or meant for the size of some object > in memory. > " > > It seems that performing casts would not be a good idea. > Should I make crop fields size_t type as in AVFrame? > Or is there a more correct way?
the type specified in the format string has to match the types passed in the argument to av_log "%"SIZE_SPECIFIER" + size_t or %d + int otherwise you have undefined behavior [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB The greatest way to live with honor in this world is to be what we pretend to be. -- Socrates
signature.asc
Description: PGP signature
_______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org https://ffmpeg.org/mailman/listinfo/ffmpeg-devel To unsubscribe, visit link above, or email ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".
