Garbled speech in Popful Mail with r700

[Former user]

Original [issue 256](https://code.google.com/p/genplus-gx/issues/detail?id=256) created by JJBunks on 2012-07-18T20:00:48.000Z:

When playing Popful Mail with GenPlus GX 1.7.0 r700, the PCM audio that plays when you meet Slick near the start of the game is glitchy. Note that this audio is fine with r683!

[Former user]

Comment # 1 originally posted by ekeeke31 on 2012-07-18T21:15:07.000Z:

Please try with r691 and then with r692 and see if the bug appeared with this revision.

[Former user]

Comment # 2 originally posted by JJBunks on 2012-07-18T23:24:45.000Z:

I'm trying to compile the revisions you've requested and am having trouble. I previously downloaded a precompiled r683 and r700, and haven't tried compiling on my own. Anyway, when trying to compile, I get these errors:

gx\_video.c In file included from c:/Users/Jeff/devkitpro/genplus691/source/gx/gx\_video.c:46:0: c:/Users/Jeff/devkitpro/portlibs/ppc/include/png.h:1:9: error: expected '=', ',', ';', 'asm' or '\_\_attribute\_\_' before '/' token c:/Users/Jeff/devkitpro/genplus691/source/gx/gx\_video.c:60:3: warning: data definition has no type or storage class [enabled by default] c:/Users/Jeff/devkitpro/genplus691/source/gx/gx\_video.c:60:3: warning: type defaults to 'int' in declaration of 'png\_image' [-Wimplicit-int] c:/Users/Jeff/devkitpro/genplus691/source/gx/gx\_video.c:1010:32: error: unknowntype name 'png\_structp' c:/Users/Jeff/devkitpro/genplus691/source/gx/gx\_video.c:1010:53: error: unknowntype name 'png\_bytep' c:/Users/Jeff/devkitpro/genplus691/source/gx/gx\_video.c:1010:69: error: unknowntype name 'png\_size\_t' c:/Users/Jeff/devkitpro/genplus691/source/gx/gx\_video.c: In function 'gxTextureOpenPNG': c:/Users/Jeff/devkitpro/genplus691/source/gx/gx\_video.c:1027:3: error: unknown type name 'png\_structp' c:/Users/Jeff/devkitpro/genplus691/source/gx/gx\_video.c:1027:3: warning: implicit declaration of function 'png\_create\_read\_struct' [-Wimplicit-function-declaration] c:/Users/Jeff/devkitpro/genplus691/source/gx/gx\_video.c:1027:48: error: 'PNG\_LIBPNG\_VER\_STRING' undeclared (first use in this function) c:/Users/Jeff/devkitpro/genplus691/source/gx/gx\_video.c:1027:48: note: each undeclared identifier is reported only once for each function it appears in c:/Users/Jeff/devkitpro/genplus691/source/gx/gx\_video.c:1031:3: error: unknown type name 'png\_infop' c:/Users/Jeff/devkitpro/genplus691/source/gx/gx\_video.c:1031:3: warning: implicit declaration of function 'png\_create\_info\_struct' [-Wimplicit-function-declaration] c:/Users/Jeff/devkitpro/genplus691/source/gx/gx\_video.c:1034:5: warning: implicit declaration of function 'png\_destroy\_read\_struct' [-Wimplicit-function-declaration] c:/Users/Jeff/devkitpro/genplus691/source/gx/gx\_video.c:1041:5: warning: statement with no effect [-Wunused-value] c:/Users/Jeff/devkitpro/genplus691/source/gx/gx\_video.c:1041:15: error: expected ';' before 'image' c:/Users/Jeff/devkitpro/genplus691/source/gx/gx\_video.c:1042:5: error: 'image' undeclared (first use in this function) c:/Users/Jeff/devkitpro/genplus691/source/gx/gx\_video.c:1046:5: warning: implicit declaration of function 'png\_set\_read\_fn' [-Wimplicit-function-declaration] c:/Users/Jeff/devkitpro/genplus691/source/gx/gx\_video.c:1046:30: error: 'png\_voidp' undeclared (first use in this function) c:/Users/Jeff/devkitpro/genplus691/source/gx/gx\_video.c:1046:41: error: expected expression before ')' token c:/Users/Jeff/devkitpro/genplus691/source/gx/gx\_video.c:1046:51: error: 'png\_read\_from\_mem' undeclared (first use in this function) c:/Users/Jeff/devkitpro/genplus691/source/gx/gx\_video.c:1051:5: error: unknown type name 'png\_byte' c:/Users/Jeff/devkitpro/genplus691/source/gx/gx\_video.c:1058:5: warning: implicit declaration of function 'png\_sig\_cmp' [-Wimplicit-function-declaration] c:/Users/Jeff/devkitpro/genplus691/source/gx/gx\_video.c:1065:5: warning: implicit declaration of function 'png\_init\_io' [-Wimplicit-function-declaration] c:/Users/Jeff/devkitpro/genplus691/source/gx/gx\_video.c:1066:5: warning: implicit declaration of function 'png\_set\_sig\_bytes' [-Wimplicit-function-declaration] c:/Users/Jeff/devkitpro/genplus691/source/gx/gx\_video.c:1075:3: warning: implicit declaration of function 'png\_read\_info' [-Wimplicit-function-declaration] c:/Users/Jeff/devkitpro/genplus691/source/gx/gx\_video.c:1078:3: warning: implicit declaration of function 'png\_get\_image\_width' [-Wimplicit-function-declaration] c:/Users/Jeff/devkitpro/genplus691/source/gx/gx\_video.c:1079:3: warning: implicit declaration of function 'png\_get\_image\_height' [-Wimplicit-function-declaration] c:/Users/Jeff/devkitpro/genplus691/source/gx/gx\_video.c:1080:3: warning: implicit declaration of function 'png\_get\_bit\_depth' [-Wimplicit-function-declaration] c:/Users/Jeff/devkitpro/genplus691/source/gx/gx\_video.c:1081:3: warning: implicit declaration of function 'png\_get\_color\_type' [-Wimplicit-function-declaration] c:/Users/Jeff/devkitpro/genplus691/source/gx/gx\_video.c:1087:24: error: 'PNG\_COLOR\_TYPE\_RGB\_ALPHA' undeclared (first use in this function) c:/Users/Jeff/devkitpro/genplus691/source/gx/gx\_video.c:1111:3: error: unknown type name 'png\_bytep' c:/Users/Jeff/devkitpro/genplus691/source/gx/gx\_video.c:1111:30: error: 'png\_bytep' undeclared (first use in this function) c:/Users/Jeff/devkitpro/genplus691/source/gx/gx\_video.c:1111:41: error: expected expression before ')' token c:/Users/Jeff/devkitpro/genplus691/source/gx/gx\_video.c:1122:21: warning: assignment makes integer from pointer without a cast [enabled by default] c:/Users/Jeff/devkitpro/genplus691/source/gx/gx\_video.c:1126:3: warning: implicit declaration of function 'png\_read\_image' [-Wimplicit-function-declaration] c:/Users/Jeff/devkitpro/genplus691/source/gx/gx\_video.c:1129:3: warning: implicit declaration of function 'png\_read\_end' [-Wimplicit-function-declaration] c:/Users/Jeff/devkitpro/genplus691/source/gx/gx\_video.c: In function 'gxTextureWritePNG': c:/Users/Jeff/devkitpro/genplus691/source/gx/gx\_video.c:1290:3: error: unknown type name 'png\_structp' c:/Users/Jeff/devkitpro/genplus691/source/gx/gx\_video.c:1290:3: warning: implicit declaration of function 'png\_create\_write\_struct' [-Wimplicit-function-declaration] c:/Users/Jeff/devkitpro/genplus691/source/gx/gx\_video.c:1290:49: error: 'PNG\_LIBPNG\_VER\_STRING' undeclared (first use in this function) c:/Users/Jeff/devkitpro/genplus691/source/gx/gx\_video.c:1298:3: error: unknown type name 'png\_infop' c:/Users/Jeff/devkitpro/genplus691/source/gx/gx\_video.c:1302:5: warning: implicit declaration of function 'png\_destroy\_write\_struct' [-Wimplicit-function-declaration] c:/Users/Jeff/devkitpro/genplus691/source/gx/gx\_video.c:1310:3: warning: implicit declaration of function 'png\_set\_IHDR' [-Wimplicit-function-declaration] c:/Users/Jeff/devkitpro/genplus691/source/gx/gx\_video.c:1310:71: error: 'PNG\_COLOR\_TYPE\_RGB\_ALPHA' undeclared (first use in this function) c:/Users/Jeff/devkitpro/genplus691/source/gx/gx\_video.c:1311:16: error: 'PNG\_INTERLACE\_NONE' undeclared (first use in this function) c:/Users/Jeff/devkitpro/genplus691/source/gx/gx\_video.c:1311:36: error: 'PNG\_COMPRESSION\_TYPE\_DEFAULT' undeclared (first use in this function) c:/Users/Jeff/devkitpro/genplus691/source/gx/gx\_video.c:1311:66: error: 'PNG\_FILTER\_TYPE\_DEFAULT' undeclared (first use in this function) c:/Users/Jeff/devkitpro/genplus691/source/gx/gx\_video.c:1314:3: error: unknown type name 'png\_bytep' c:/Users/Jeff/devkitpro/genplus691/source/gx/gx\_video.c:1314:30: error: 'png\_bytep' undeclared (first use in this function) c:/Users/Jeff/devkitpro/genplus691/source/gx/gx\_video.c:1314:41: error: expected expression before ')' token c:/Users/Jeff/devkitpro/genplus691/source/gx/gx\_video.c:1325:21: warning: assignment makes integer from pointer without a cast [enabled by default] c:/Users/Jeff/devkitpro/genplus691/source/gx/gx\_video.c:1329:3: warning: implicit declaration of function 'png\_set\_rows' [-Wimplicit-function-declaration] c:/Users/Jeff/devkitpro/genplus691/source/gx/gx\_video.c:1332:3: warning: implicit declaration of function 'png\_write\_png' [-Wimplicit-function-declaration] c:/Users/Jeff/devkitpro/genplus691/source/gx/gx\_video.c:1332:34: error: 'PNG\_TRANSFORM\_IDENTITY' undeclared (first use in this function) c:/Users/Jeff/devkitpro/genplus691/source/gx/gx\_video.c:1335:3: warning: implicit declaration of function 'png\_write\_end' [-Wimplicit-function-declaration] make[1]: \*\*\* [gx\_video.o] Error 1 make: \*\*\* [build\_wii] Error 2

I assume the problem is with the "libpng-1.5.4-ppc" file I downloaded. I'm pretty sure I got the correct version. Can you lend me some help with this?

[Former user]

Comment # 3 originally posted by ekeeke31 on 2012-07-19T11:31:24.000Z:

Have you followed the steps in the Compiling section ? Seems like you are missing a few libraries or included files, make sure libogc is properly installed as well.

Anyway, about the issue, could you upload a savestate just before it happens so i can quickly try to reproduce it when i have some time ?

[Former user]

Comment # 4 originally posted by JJBunks on 2012-07-19T16:11:52.000Z:

These compiling errors are irritating! I have installed devkitPro with the msys tools, devkitPPC in devkitpro
devkitPPC, libogc and libfat in devkitpro
libogc, and zlib, libpng, and tremor in devkitpro
portlibs
ppc. Am I missing something here?

Here is a savestate right before the audio in question plays. Just walk to the right and it will start (I saved this with the 1.7.0 binary on this Google Code site).

[Former user]

Comment # 5 originally posted by ekeeke31 on 2012-07-20T11:34:41.000Z:

Thanks for the savestate.

No idea why it wouldn't compile, i do not have updated libpng for ages and compile under windows. The error seems to be in included file png.h (line 9), all following errors are then due to missing libpng declarations since that file was not interpreted correctly. Check comments in Compiling section as i think an user also had some issues with badly extracted library files.

[Former user]

Comment # 6 originally posted by JJBunks on 2012-07-21T04:12:29.000Z:

I have some news: I downgraded libpng from 1.5.4 to 1.4.1 and can now compile GenPlus GX with no errors (finally!). Why was the newer version giving me trouble? The builds seem to function without error, so using the older version doesn't seem to be an issue.

So, I tested r691 and r692. r691 plays the PCM audio perfectly and r692 has the glitchy stuttering issue, so whatever changes were made to pcm.c at that point negatively affected the game, but fixed others like Panic!/Switch.

[Former user]

Comment # 7 originally posted by ekeeke31 on 2012-07-21T09:02:08.000Z:

Thank you for testing. I have no way no test or code anything for the time being but will try to found what went wrong in these changes. I noticed screen freezing issues in Snatcher (old factory section) which seem ro be related to pcm emulation as well.

As for libpng, i don't know what is wrong with the last version and don't really care honestly, it's only used to display menu images anyway, old versions are perfectly fine. The error seems to indicate a problem in png.h, not an incompatibility from the emulator with libpng, so maybe it is not compatible with devkitppc or it's missing something.

[Former user]

Comment # 8 originally posted by ekeeke31 on 2012-07-21T17:12:33.000Z:

It was actually a stupid C coding mistake I made, should be fixed in r701.

[Former user]

Comment # 9 originally posted by ekeeke31 on 2012-07-22T17:48:08.000Z: