[pcre-dev] [Bug 1295] add 32-bit library

Page principale
Supprimer ce message
Auteur: Christian Persch
Date:  
À: pcre-dev
Sujet: [pcre-dev] [Bug 1295] add 32-bit library
------- 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 #17 from Christian Persch (GNOME) <chpe@???> 2012-10-17 19:07:00 ---
(In reply to comment #16)
>   . 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.


I think some already pre-existed... I tried compiling with -Wsign-compare
-Wconversion to find places where 32-bit characters might get mangled, but
there were simply too many warnings, so I gave up :-) It would be a good idea
to do a sweep and fix them all...

>   . 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.)


I try to avoid them usually; I only use them for 'fixme' style comments I
intend to remove again. It's possible I forgot some.


--
Configure bugmail: http://bugs.exim.org/userprefs.cgi?tab=email