[MPlayer-dev-eng] libvo: GGI driver update
Diego Biurrun
diego at biurrun.de
Sun Sep 11 10:05:54 CEST 2005
On Sun, Sep 11, 2005 at 09:29:35AM +0200, Christoph Egger wrote:
>
> > I believe Reimar already made a comment about it, but why did you move
> > draw_slice up? It's hard to see what exactly changed. Right now it's
> > between flip_page and query_format, so please keep it there for clarity.
>
> The reason is logical order. With this move, draw_frame(), draw_osd() and
> draw_slice() are grouped together.
>
> With this move you have some rough "sections" in vo_ggi.c:
>
> 1. Datatypes
> 2. Configuration
> 3. rendering (get_image, draw_*)
> 4. (un)initialization
> 5. event handling
>
> If you count this move as cosmetic and don't like it because of this,
> I'll undo it.
The move is cosmetic. If you wish to move the function, please send a
separate patch.
Diego
More information about the MPlayer-dev-eng
mailing list