Hi Graham, [Side note: maybe we can stop using the @packages.debian.org address, I don't think anybody else is listening.] On 25-05-13 21:08, Graham Inggs wrote: > Thanks for uploading! Thank you for the nice cooperation so far, it has been a pleasure. > I will file the bugs against the reverse dependencies of lesstif2-dev in > Debian. Good, but I think it is also good to wait until all architectures have build successfully. > I will also file one in Ubuntu to remove openmotif and replace with motif. Good. (Same holds as above. And please let me know the bug number or subscribe me (paul-climbing)) I believe the following is a minor thing, but I wanted to let you know anyway. I am looking at [1], which is accessible from the top of the QA page [2] and see that we have an interesting issue. The first item is mentioned as an error on ia64 and powerpc: DataF.c:4581:35: warning: array subscript is above array bounds [-Warray-bounds] I don't really think it is a problem at this location, but looking at the code I see the text: /* This routine accepts an array of wchar_t's containing wchar encodings * of whitespace characters (and the number of array elements), comparing * the wide character passed to each element of the array. If a match * is found, we got a white space. This routine exists only because * iswspace(3c) is not yet standard. If a system has isw* available, * calls to this routine should be changed to iswspace(3c) (and callers * should delete initialization of the array), and this routine should * be deleted. Its a stop gap measure to avoid allocating an instance * variable for the white_space array and/or declaring a widget wide * global for the data and using a macro. Its ugly, but it works and * in the long run will be replaced by standard functionality. */ Maybe we should think about patching motif (and sending that upstream obviously) to use iswspace (part of libc6). What do you think? It would mean that motif because more locale aware with respect to whitespaces, because now it only declares " ", \t and \n, while there are more whitespaces. Paul [1] https://buildd.debian.org/~brlink/packages/m/motif.html [2] http://packages.qa.debian.org/m/motif.html
Attachment:
signature.asc
Description: OpenPGP digital signature