Re: [hatari-devel] Synching host mouse position to Atari mouse position

[ Thread Index | Date Index | More lists.tuxfamily.org/hatari-devel Archives ]


Am Sun, 1 Dec 2019 00:59:26 +0200
schrieb Eero Tamminen <oak@xxxxxxxxxxxxxx>:

> Hi,
> 
> On 11/30/19 9:26 AM, Thomas Huth wrote:
> > Am Sat, 30 Nov 2019 02:04:36 +0200
> > schrieb Eero Tamminen <oak@xxxxxxxxxxxxxx>:
> > 
> >> It includes also output for SDL window events
> >> which one can enable by increasing log level.
> > 
> > I don't quite see the point why Hatari needs a verbose string for
> > each Window event ... seems a little bit too much code for me here
> > ... can't you simply print out the number? If someone is interested
> > in the events, they can look up the number in the SDL_video.h
> > header instead.
> 
> Thanks, attached is smaller patch (still
> SDL2-only).

Looks fine at a first glance, but please use "/* fall through */"
instead of "/* pass-through */", so that the comment can be used to
silence the warnings from -Wimplicit-fallthrough=2.

 Thanks,
  Thomas



Mail converted by MHonArc 2.6.19+ http://listengine.tuxfamily.org/