25 Dec
2010
25 Dec
'10
4:29 p.m.
Hi, I'd like a feedback on this patch: there is still a "main" label in mplayer.c. This is patch to get rid of it. The code first check if there is a video context (is it the correct name for sh_video?). If there is, the video chain is reset. After that, if neither sh_video nor sh_audio are set, it goes to the next file. Then, two things are done, but only if there was a video context at first, so I just moved this in the "main" code where sh_video is checked. It seems no previous code depends on the spu state and the font load flag. Btw, mplayer.c really needs at least a reindent; the current state is quiet a mess… -- Clément B. Not sent from a jesusPhone.