Re: [Pdl-devel] cleanups

2016-06-07 Thread Derek Lamb
8, 7, 6, 5, and 4 (for Gaussian) are fixed in the clang_warnings branch. I have solutions for 4 (for FFT) and 2 (the abs/labs part), but I'm not sure how portable they are. See below. I'll need to think some more about 2 (the Anyval part). # inc is of type PDL_Indx, on my

Re: [Pdl-devel] cleanups

2016-06-04 Thread Chris Marshall
Hi Karl- I suggest opening a ticket with the information on platform, compiler, and the log of issues. While warnings are often only annoying, they can indicate a potential problem in general or a platform specific one. Unfortunately, without a full, multi-platform regression test suite, it can

Re: [Pdl-devel] cleanups

2016-06-03 Thread Derek Lamb
1) was addressed in the deprecate-finite branch that was recently merged into master post-2.016. 3) is addressed in the types-printconv branch that has not yet been merged—needs some code review to make sure it will pass on 32-bit and 64-bit platforms with different values of ivsize, etc. It h

[Pdl-devel] cleanups

2016-06-03 Thread Karl Glazebrook
Any reason why these 8 build warnings in PDL aren’t trivial to fix? [e.g. would it break other architectures...] Yes I’ve been ignoring them for a long time, but perhaps it is time for a spring clean? - Karl —-- 1) pdlcore.c:1292:86: warnin