Marvin Scholz (HE12025-06-25): > Would you be fine with just the removal of the messing with > the AVDictionary entries then, leaving the macros in place, > essentially removing STEAL_OPTION and doing a copy in CONSUME_OPTION? > > IMHO saving two copies of a string does not justify abusing the > AVDictionary API in such a way. This isnt a hot code path either > where this would make sense...
It is not just a matter of saving a few cycles. What you propose requires writing more code, including error checks and an occasion for failure. If it was new code, I would consider it, but changing existing code that has been working for years to make it more verbose, more failure prone and less efficient, no, thanks. Let us just document that it is a valid use. Regards, -- Nicolas George _______________________________________________ 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".
