> You should restrict the line length here, just for consistency with the > rest of this document. (It doesn't affect the formatting of the output > documents, obviously.) > What would be a good line length for the docs to aim for? 75?
> > + unsigned logical_processors; > > The libsvtav1 API defines this as uint32_t, so I believe you should > mirror that. > Changed locally, didn't see any other usage of stdint.h types in the file so I just followed along > > + AV_OPT_TYPE_INT, { .i64 = 0 }, 0, INT_MAX, VE }, > > Probably UINT_MAX, though I doubt that such a number of processors will > be reached. ;-) I don't know whether there's another natural limit > within libsvtav1. > Changed locally, hopefully, one day this limit will actually have a use (crossed fingers for AMD or some potentially arm or riscv). The only internal limits I am aware of is the return type of sysconf being shortened to uint32_t and windows' DWORD length Thanks, Chris _______________________________________________ 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".