Sun, 02 Sep 2012 19:37:36 -0400Removed a FIXME; RemoteIO is correct for iOS.
Ryan C. Gordon [Sun, 02 Sep 2012 19:37:36 -0400] rev 6413
Removed a FIXME; RemoteIO is correct for iOS.

Sat, 01 Sep 2012 00:25:17 -0700Added testaudioinfo to the build.
Ryan C. Gordon [Sat, 01 Sep 2012 00:25:17 -0700] rev 6412
Added testaudioinfo to the build.

Sun, 02 Sep 2012 16:05:29 -0700Use fast path for RGB 565 -> 32-bit XRGB 8888 SDL-1.2
Sam Lantinga [Sun, 02 Sep 2012 16:05:29 -0700] rev 6411
Use fast path for RGB 565 -> 32-bit XRGB 8888

Hello Sam,
while profiling ScummVM I noticed it was making use of the generic
BlitNToN blitter, which struck me as odd because it should be a very
classical codepath.
After investigating, I saw that in the blit op chooser:

{ 0x0000F800,0x000007E0,0x0000001F, 4, 0x00FF0000,0x0000FF00,0x000000FF,
0, NULL, Blit_RGB565_ARGB8888, SET_ALPHA },
{ 0x0000F800,0x000007E0,0x0000001F, 4, 0x000000FF,0x0000FF00,0x00FF0000,
0, NULL, Blit_RGB565_ABGR8888, SET_ALPHA },
{ 0x0000F800,0x000007E0,0x0000001F, 4, 0xFF000000,0x00FF0000,0x0000FF00,
0, NULL, Blit_RGB565_RGBA8888, SET_ALPHA },
{ 0x0000F800,0x000007E0,0x0000001F, 4, 0x0000FF00,0x00FF0000,0xFF000000,
0, NULL, Blit_RGB565_BGRA8888, SET_ALPHA },

Couldn't the optimized versions be used for NO_ALPHA too? I take it
that the resulting alpha component can be undefined as it should never
be used.
I tried this (see attached patch) and it worked perfectly (and
therefore faster) on ScummVM but there might be a trick (I'm not
expert at the semantics of SDL, ie NO_ALPHA, SET_ALPHA and COPY_ALPHA
there).
What do you think?

Cheers,
Bertrand

Sun, 02 Sep 2012 16:03:56 -0700Use fast path for RGB 565 -> 32-bit XRGB 8888
Sam Lantinga [Sun, 02 Sep 2012 16:03:56 -0700] rev 6410
Use fast path for RGB 565 -> 32-bit XRGB 8888

Hello Sam,
while profiling ScummVM I noticed it was making use of the generic
BlitNToN blitter, which struck me as odd because it should be a very
classical codepath.
After investigating, I saw that in the blit op chooser:

{ 0x0000F800,0x000007E0,0x0000001F, 4, 0x00FF0000,0x0000FF00,0x000000FF,
0, NULL, Blit_RGB565_ARGB8888, SET_ALPHA },
{ 0x0000F800,0x000007E0,0x0000001F, 4, 0x000000FF,0x0000FF00,0x00FF0000,
0, NULL, Blit_RGB565_ABGR8888, SET_ALPHA },
{ 0x0000F800,0x000007E0,0x0000001F, 4, 0xFF000000,0x00FF0000,0x0000FF00,
0, NULL, Blit_RGB565_RGBA8888, SET_ALPHA },
{ 0x0000F800,0x000007E0,0x0000001F, 4, 0x0000FF00,0x00FF0000,0xFF000000,
0, NULL, Blit_RGB565_BGRA8888, SET_ALPHA },

Couldn't the optimized versions be used for NO_ALPHA too? I take it
that the resulting alpha component can be undefined as it should never
be used.
I tried this (see attached patch) and it worked perfectly (and
therefore faster) on ScummVM but there might be a trick (I'm not
expert at the semantics of SDL, ie NO_ALPHA, SET_ALPHA and COPY_ALPHA
there).
What do you think?

Cheers,
Bertrand

Thu, 30 Aug 2012 12:58:58 -0700Fixed compiler warning.
Ryan C. Gordon [Thu, 30 Aug 2012 12:58:58 -0700] rev 6409
Fixed compiler warning.

Sun, 26 Aug 2012 20:27:25 -0400Removed AC_GNU_SOURCE from SDL2's configure script.
Ryan C. Gordon [Sun, 26 Aug 2012 20:27:25 -0400] rev 6408
Removed AC_GNU_SOURCE from SDL2's configure script.

Sat, 25 Aug 2012 22:21:16 -0400Fixed compiler warning on some versions of GCC.
Ryan C. Gordon [Sat, 25 Aug 2012 22:21:16 -0400] rev 6407
Fixed compiler warning on some versions of GCC.

Sat, 25 Aug 2012 16:49:05 -0400Fixed wglShareLists() call that used the wrong variable.
Ryan C. Gordon [Sat, 25 Aug 2012 16:49:05 -0400] rev 6406
Fixed wglShareLists() call that used the wrong variable.

Sat, 25 Aug 2012 16:46:34 -0400Removed _GNU_SOURCE define. We'll see if we need it. SDL-1.2
Ryan C. Gordon [Sat, 25 Aug 2012 16:46:34 -0400] rev 6405
Removed _GNU_SOURCE define. We'll see if we need it.

Fri, 24 Aug 2012 19:39:51 -0400Whoops, removed wrong variable.
Ryan C. Gordon [Fri, 24 Aug 2012 19:39:51 -0400] rev 6404
Whoops, removed wrong variable.