| 2 | | I think the best solution is to just change the parameter type of "number", currently int, to uint64_t. The function av_get_frame_filename2 is called in many places with an int "number" argument, except for via the frame_pts codepath, where it's a uint64_t "number" argument. It seems safer to me to cast int into uint64_t rather than the other way round, and I don't foresee any side-effects, though I'm inexperienced with C. Shall I try to submit that as a patch? |
| | 2 | I think the best solution is to just change the parameter type of "number", currently int, to int64_t. The function av_get_frame_filename2 is called in many places with an int "number" argument, except for via the frame_pts codepath, where it's a int64_t "number" argument. It seems safer to me to cast int into int64_t rather than the other way round, and I don't foresee any side-effects, though I'm inexperienced with C. Shall I try to submit that as a patch? |