michaelni changed the topic of #ffmpeg-devel to: Welcome to the FFmpeg development channel | Questions about using FFmpeg or developing with libav* libs should be asked in #ffmpeg | This channel is publicly logged | FFmpeg 7.0.2 has been released! | Please read ffmpeg.org/developer.html#Code-of-conduct
arch1t3cht5 has joined #ffmpeg-devel
arch1t3cht has quit [Ping timeout: 260 seconds]
arch1t3cht5 is now known as arch1t3cht
cone-606 has quit [Quit: transmission timeout]
lexano has quit [Ping timeout: 252 seconds]
thilo has quit [Ping timeout: 246 seconds]
thilo has joined #ffmpeg-devel
IndecisiveTurtle has quit [Remote host closed the connection]
kasper93_ has joined #ffmpeg-devel
kasper93 has quit [Ping timeout: 260 seconds]
Kei_N_ has quit [Ping timeout: 276 seconds]
Kei_N has joined #ffmpeg-devel
Kei_N has quit [Ping timeout: 260 seconds]
Kei_N has joined #ffmpeg-devel
tufei has joined #ffmpeg-devel
Martchus has joined #ffmpeg-devel
Martchus_ has quit [Ping timeout: 246 seconds]
jamrial_ has quit []
SystemError has quit [Read error: Connection reset by peer]
SystemError has joined #ffmpeg-devel
SystemError has quit [Remote host closed the connection]
llyyr has quit [Quit: Quit]
llyyr has joined #ffmpeg-devel
llyyr has quit [Changing host]
llyyr has joined #ffmpeg-devel
SystemError has joined #ffmpeg-devel
HarshK23 has joined #ffmpeg-devel
j45_ has joined #ffmpeg-devel
j45 has quit [Ping timeout: 260 seconds]
j45_ is now known as j45
j45 has joined #ffmpeg-devel
j45 has quit [Changing host]
mkver has quit [Ping timeout: 246 seconds]
kasper93_ is now known as kasper93
kasper93_ has joined #ffmpeg-devel
odrling_ has joined #ffmpeg-devel
uartie_ has joined #ffmpeg-devel
microchip__ has joined #ffmpeg-devel
Venemo_ has joined #ffmpeg-devel
novaphoenix0 has joined #ffmpeg-devel
thilo_ has joined #ffmpeg-devel
thilo_ has quit [Changing host]
thilo_ has joined #ffmpeg-devel
kasper93 has quit [*.net *.split]
thilo has quit [*.net *.split]
odrling has quit [*.net *.split]
microchip_ has quit [*.net *.split]
novaphoenix has quit [*.net *.split]
uartie has quit [*.net *.split]
Venemo has quit [*.net *.split]
odrling_ is now known as odrling
uartie_ is now known as uartie
lemourin has quit [Read error: Connection reset by peer]
lemourin has joined #ffmpeg-devel
microchip__ is now known as microchip_
Sean_McG has joined #ffmpeg-devel
IndecisiveTurtle has joined #ffmpeg-devel
Krowl has joined #ffmpeg-devel
rvalue has quit [Read error: Connection reset by peer]
rvalue has joined #ffmpeg-devel
Krowl has quit [Read error: Connection reset by peer]
lexano has joined #ffmpeg-devel
sepro has quit [Quit: Ping timeout (120 seconds)]
sepro has joined #ffmpeg-devel
microchip_ has quit [Quit: There is no spoon!]
microchip_ has joined #ffmpeg-devel
SystemError has quit [Remote host closed the connection]
SystemError has joined #ffmpeg-devel
<jdarnley> Can a stream can have only 1 "disposition"?
<Lynne> it can even have no disposition
<jdarnley> 0 or 1 then
<jdarnley> "Stream disposition - a combination of AV_DISPOSITION_* flags." so no
Traneptora has quit [Quit: Quit]
Krowl has joined #ffmpeg-devel
kasper93_ is now known as kasper93
elvis_a_presley has quit [Quit: smoke-bomb ; grapple-hook]
elvis_a_presley has joined #ffmpeg-devel
mkver has joined #ffmpeg-devel
IndecisiveTurtle has quit [Ping timeout: 265 seconds]
SystemError has quit [Remote host closed the connection]
SystemError has joined #ffmpeg-devel
<Sean_McG> Marth64[m]: I have finally acquired a PowerMac G5. Arrived in time for my birthday yesterday, but it needs a major clean and a new battery among other things. It'll be nice to move my FATE nodes to it from the QEMU setup I have now.
Traneptora has joined #ffmpeg-devel
IndecisiveTurtle has joined #ffmpeg-devel
marcj has quit [Quit: ZNC 1.7.5+deb4 - https://znc.in]
<BtbN> elenril: https://github.com/FFmpeg/FFmpeg/commit/450a3f58edb22d28912a5e65dc08d9e2fb805066 if the new function here runs through to the end without error, it'll return an uninitialized "ret" value. Is it missing a "return 0;" before that fail-Label, or should ret be initialized or something?
<elenril> huh, I remember initializing it
<elenril> BtbN: it should be initialized to 0
<BtbN> Wasn't sure due to the opt_free_array
<elenril> returning before fail also works, since new_elems gets set to NULL in both branches
<BtbN> i.e. if that should always run, or only on error
<elenril> it does nothing on success, so either is fine
<BtbN> Should I just quickly push a patch, or do you want to do so?
Krowl has quit [Read error: Connection reset by peer]
<BtbN> I'd just initialize ret to 0
rvalue- has joined #ffmpeg-devel
<elenril> sure, go ahead
<elenril> less work for me on a saturday evening :)
rvalue has quit [Ping timeout: 252 seconds]
cone-236 has joined #ffmpeg-devel
<cone-236> ffmpeg Timo Rothenpieler master:7e35aeda03e7: lavu/opt: properly initialize av_opt_set_array return value
<elenril> thanks
rvalue- is now known as rvalue
<ePirat> BtbN, see the patch I just sent too
<ePirat> thats needed as well else you return success in an error case now with your patch
<BtbN> I didn't even see that, yeah, shadowing like that really should be an error imo
<BtbN> Isn't there a gcc warning flag for that?
<ePirat> BtbN, while at it please also merge my other patch for av_opt_get_array (lavu/opt: Fix return of uninitialised value)
<ePirat> same issue, different place
<BtbN> The patch looks fine. You can't just push?
<ePirat> no
<ePirat> it seems people assume I do so especially trivial patches like this lay on the ML forever until I annoy someone to merge for me :D
<BtbN> I only see one patch, am I missing one?
<BtbN> "lavu/opt: av_opt_set_array: fix uninitialised return"
<ePirat> I love the mailing list
<ePirat> where stuff goes to never to be found again
<BtbN> oh, it's just older than expected
<ePirat> I assumed elenril had merged it
<ePirat> but I just checked earlier and it seems he did not :D
<cone-236> ffmpeg Marvin Scholz master:b4e64b86ad92: lavu/opt: av_opt_set_array: fix uninitialised return
<cone-236> ffmpeg Marvin Scholz master:8a314e3f11dc: lavu/opt: Fix return of uninitialised value
<ePirat> thank you
<BtbN> good god... I assed -Werror=shadow to the cflags, and it's bad
<BtbN> *added
<ePirat> yeah
<ePirat> we do that a lot iirc
<BtbN> I'm surprised it's not a default-warning
<BtbN> this is just never good practice
<JEEB> aye
<JEEB> I wonder if I should attempt to add the array size checker type hints at some point
<JEEB> that seems like a useful check as well, and kernel seems to be utilizing it
<ePirat> JEEB, contemplated the same but we really can only do so for internal APIs, but still might be worth it
paulk-bis has joined #ffmpeg-devel
paulk has quit [Read error: Connection reset by peer]
<Marth64[m]> Sean_McG: sorry for the slow response. congratulations! wishing your retro rig a long life
<JEEB> ePirat: then there's some new recommendation for various compilation flags, should look if that has interesting bits there
marcj has joined #ffmpeg-devel
marcj has quit [Quit: ZNC 1.7.5+deb4 - https://znc.in]
marcj has joined #ffmpeg-devel
marcj has quit [Quit: ZNC 1.7.5+deb4 - https://znc.in]
marcj has joined #ffmpeg-devel
marcj has quit [Quit: ZNC 1.7.5+deb4 - https://znc.in]
paulk-bis has quit [Quit: WeeChat 3.0]
paulk has joined #ffmpeg-devel
paulk has joined #ffmpeg-devel
paulk has quit [Changing host]
mkver has quit [Ping timeout: 260 seconds]
marcj has joined #ffmpeg-devel
mkver has joined #ffmpeg-devel
another| has quit [Remote host closed the connection]
cone-236 has quit [Quit: transmission timeout]
Traneptora has quit [Quit: Quit]
Traneptora has joined #ffmpeg-devel
Traneptora has quit [Remote host closed the connection]
another has joined #ffmpeg-devel
BtbN has quit [Remote host closed the connection]
BtbN has joined #ffmpeg-devel
BtbN has quit [Remote host closed the connection]
BtbN has joined #ffmpeg-devel
BtbN has quit [Remote host closed the connection]
BtbN has joined #ffmpeg-devel
another is now known as another|
uau has quit [Quit: ZNC 1.9.1+deb1 - https://znc.in]
uau has joined #ffmpeg-devel
Traneptora has joined #ffmpeg-devel
jamrial has joined #ffmpeg-devel