[vlc-devel] Re: [PATCH] bugfix: mosaic-[hv]border reversed

[Antoine Cellerier]

On Tue, Oct 03, 2006, Thomas Parmelan wrote:
> I think I found 2 bugs related to mosaic-hborder and mosaic-vborder.
> 1. The help says hborder means horizontal border and vborder means
>    vertical border, whereas the code uses hborder as border height and
>    vborder as border width!
> 2. (minor) In a debug message of the MosaicCallback() function,
>    p_sys->i_vborder is used instead of p_sys->i_hborder when parsing the
>    mosaic-hborder option.
> The attached patch (against trunk) fixes the code to match the help
> text, but I can propose a patch the other way around if people think it
> is better (but I would suggest renaming hborder to borderh and vborder
> to borderw (NOT borderv) in this case!).

The borderh/borderw names seem better. Could you send a patch for those?

Thanks,

-- 
Antoine Cellerier
dionoea

-- 
This is the vlc-devel mailing-list, see http://www.videolan.org/vlc/
To unsubscribe, please read http://developers.videolan.org/lists.html