I'd also prefer to use std::abs (and std::????) whenever possible. Hans
From: Bill Lorensen <[email protected]<mailto:[email protected]>> Date: Wednesday, October 31, 2012 9:55 PM To: Sean McBride <[email protected]<mailto:[email protected]>> Cc: "[email protected]<mailto:[email protected]>" <[email protected]<mailto:[email protected]>>, ITK <[email protected]<mailto:[email protected]>> Subject: Re: [Insight-developers] API change of vnl_math to namespace Is there any reason for us to be using vnl_math? On Wed, Oct 31, 2012 at 5:02 PM, Sean McBride <[email protected]<mailto:[email protected]>> wrote: On Wed, 31 Oct 2012 20:47:57 +0000, Matt McCormick said: >Hi, > >Recently, changes were made in vnl_math.h so that vnl_math_abs is now >vnl_math::abs, etc. > >This is causing pain in ITK :-(. > >What is the recommended way to deal with this change in a >backwards-compatible way? I could not find pre-processor version >definitions in core/vnl/vnl_config.h or core/vxl_config.h This is partly my fault... and is ironic because the change was meant to *help* ITK. :( I submitted a patch to the vxl folks based on this: <http://review.source.kitware.com/#/c/6025/> see the thread "Patch from ITK (Avoid C++0x constexpr error)" on the vxl list. After some discussion about whether to accept the patch or not, it was accepted (but expanded on!) and committed in r35854 (and perhaps others). I'm not sure why Peter made even more extensive changes... Peter? Cheers, -- ____________________________________________________________ Sean McBride, B. Eng [email protected]<mailto:[email protected]> Rogue Research www.rogue-research.com<http://www.rogue-research.com> Mac Software Developer Montréal, Québec, Canada _______________________________________________ Powered by www.kitware.com<http://www.kitware.com> Visit other Kitware open-source projects at http://www.kitware.com/opensource/opensource.html Kitware offers ITK Training Courses, for more information visit: http://kitware.com/products/protraining.php Please keep messages on-topic and check the ITK FAQ at: http://www.itk.org/Wiki/ITK_FAQ Follow this link to subscribe/unsubscribe: http://www.itk.org/mailman/listinfo/insight-developers -- Unpaid intern in BillsBasement at noware dot com ________________________________ Notice: This UI Health Care e-mail (including attachments) is covered by the Electronic Communications Privacy Act, 18 U.S.C. 2510-2521, is confidential and may be legally privileged. If you are not the intended recipient, you are hereby notified that any retention, dissemination, distribution, or copying of this communication is strictly prohibited. Please reply to the sender that you have received the message in error, then delete it. Thank you. ________________________________
_______________________________________________ Powered by www.kitware.com Visit other Kitware open-source projects at http://www.kitware.com/opensource/opensource.html Kitware offers ITK Training Courses, for more information visit: http://kitware.com/products/protraining.php Please keep messages on-topic and check the ITK FAQ at: http://www.itk.org/Wiki/ITK_FAQ Follow this link to subscribe/unsubscribe: http://www.itk.org/mailman/listinfo/insight-developers
