------- You are receiving this mail because: ------- You are on the CC list for the bug.
http://bugs.exim.org/show_bug.cgi?id=1295 --- Comment #16 from Philip Hazel <[email protected]> 2012-10-17 16:33:44 --- On Tue, 16 Oct 2012, Christian Persch wrote: > I've tested this extensively, using different configure options and compile > flags (-O0, -O2), on x86. It's quite possible there are problems either in new > code, or regressions, on other architectures; so it would be great if everyone > could test this! :-) This is excellent work! Thank you for all the effort. I have found some issues, which I am happy to work on myself. They are all trivialities. 1. It fails to build if you select only 8 and 16 bit libraries: the variables cn32ptr and gn32ptr are not defined but are used in pcretest. 2. When selecting all 3 libraries it builds and runs the tests, but there are a number of compiler warnings, typically . Signed/unsigned comparisons. I know the source is a complete mess in this regard (all my fault for not getting it right at the start) but a few casts should fix it. . C++ style comments: I have avoided using these just in case somebody with a non-C++ compiler wants to compile it. (I have no idea, for example, if the z/OS compiler recognizes C++ comments.) . When not compiling 32-bit, there were a number of warnings like this: pcretest.c:1953:3: warning: implicit declaration of function 'PCHARSV32' pcretest.c:1953:3: warning: nested extern declaration of 'PCHARSV32' > Things remaining to do: > * update RunTest.bat. It appears this wasn't updated in a while (missing tests > 21 and 22 already), and I'm not confident in writing this language, so I left > it be. I am not confident about that either. Hopefully one of our Windows users will help when we make a Release Candidate. > * test CMake builds I will test this once I have got rid of all the compiler warnings. > * make JIT support the masking of high bits in UTF mode That's Zoltán's problem. :-) Philip -- Configure bugmail: http://bugs.exim.org/userprefs.cgi?tab=email -- ## List details at https://lists.exim.org/mailman/listinfo/pcre-dev
