[MPlayer-dev-eng] [PATCH] fix aspect stuff in widescreen monitors

Onur Küçük onur at delipenguen.net
Mon Apr 16 15:20:14 CEST 2007

On Mon, 16 Apr 2007 10:15:04 +0200
Reimar Doeffinger <Reimar.Doeffinger at stud.uni-karlsruhe.de> wrote:

> Certainly not, the patch is nonsense, it duplicates the code for
> monitorpixelaspect.

 I managed the patch that way, to respect the man page. In man page it
says "monitorpixelaspect overrides monitoraspect".

> It can be done by changing only 3 lines without adding any code
> (changing default values of monitoraspect and monitorpixelaspect and
> changing "if (monitorpixelaspect)" to "if (!monitoraspect)".

 I have made another patch according to the way you described. Though
with this change, monitoraspect overrides monitorpixelaspect so man page
needs to be updated too.

 If this is the way you prefer I can extend the patch to include a man
page change. If not, please let me know how you prefer it.

 Onur Küçük                                      Knowledge speaks,   
 <onur.--.-.delipenguen.net>                     but wisdom listens  

-------------- next part --------------
A non-text attachment was scrubbed...
Name: respect_aspect_2.patch
Type: application/octet-stream
Size: 559 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/mplayer-dev-eng/attachments/20070416/dc7334e6/attachment.obj>

More information about the MPlayer-dev-eng mailing list