[Ffmpeg-devel] [PATCH] Auto-resize ffplay video
Steven Sheehy
steven.sheehy
Sat Dec 30 00:07:14 CET 2006
Michael Niedermayer wrote:
> Hi
>
> On Tue, Dec 26, 2006 at 03:12:38PM -0600, Steven Sheehy wrote:
>> On Sat, Dec 23, 2006 at 15:20:36 CET 2006, Michael Niedermayer wrote:
>>> seperate issue seperate patch please
>> I could've characterized this patch more generically by saying it resolves
>> the improper resizing of the video window, if you prefer. In addition, if I
>> were to make them separate they would still depend on each other since the
>> patch to auto-resize the output modifies text before the exiting from full
>> screen bug. Nevertheless, here's the patch for just this issue:
>>
>
> [mangled unuseable patch]
>
>>> screen_width / screen_height should either be set to the dimensions of
>>> the video or should be at 0 by default and then be overridden by
>>> the video w/h if they are 0 where used
>> I've tried this, but I've found that if you pass a width and height of 0 to
>> the initial SDL_SetVideoMode(), it causes the video window to be maximized
>> and future SDL_SetVideoMode()'s to be ignored (SDL bug?). screen_width /
>> screen_height cannot be set to the video dimensions at declaration time or
>> in main() since they can only be determined at a much later point in the
>> program when alloc_picture() is called. If have any suggestions to overcome
>> these issues, I'd be grateful.
>
> what about calling SDL_SetVideoMode() after av_find_stream_info() ?
>
I followed your advice and basically moved the initial call to
SDL_SetVideoMode() from main() to decode_thread() after av_find_stream_info().
This works perfectly for me, but I'm not sure if this is how you want it. Let me
know if I need to tweak it more.
> [...]
>
>
> ------------------------------------------------------------------------
>
> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel at mplayerhq.hu
> http://lists.mplayerhq.hu/mailman/listinfo/ffmpeg-devel
-------------- next part --------------
A non-text attachment was scrubbed...
Name: resize.diff
Type: text/x-patch
Size: 4166 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20061229/ebd4971b/attachment.bin>
More information about the ffmpeg-devel
mailing list