[MPlayer-dev-eng] [PATCH] libvo: GGI driver update
Christoph Egger
Christoph_Egger at gmx.de
Thu Sep 8 21:09:32 CEST 2005
> Hi,
> On Thu, Sep 08, 2005 at 08:10:07PM +0200, Christoph Egger wrote:
> > > How about addressing Reimar's review now? He had some questions and
> > > suggested some improvements.
> >
> > I did. I changed the if in get_image() as he suggested.
> >
> > And to answer his question about ggi_conf.async - this variable
> > was only used for debugging and thus removed. The actual asynchronous
> > mode is set via ggiSetFlags()
>
> That leaves the questions why you moved some things (draw_slice
> function, async setting, ), and I consider this cosmetics:
> - if ((char *) arg) {
> + if (arg != NULL) {
> int i = 0;
> ggi_conf.driver = strdup(arg);
> - while (ggi_conf.driver[i]) {
> + while (ggi_conf.driver[i] != '\0') {
Why do you call this a cosmetic change?
The compilers parser, syntax analyser, etc. handles this differently.
And the internal syntax tree is also different.
the 'arg' is a const char * by function parameter.
Casting it to char * disqualifies this datatype for no reason.
I call this a bug. Don't know what you ever learned about
type safety (in C, pascal, java, etc.).
--
Greetings,
Christoph
Lust, ein paar Euro nebenbei zu verdienen? Ohne Kosten, ohne Risiko!
Satte Provisionen für GMX Partner: http://www.gmx.net/de/go/partner
More information about the MPlayer-dev-eng
mailing list