From: Gehua Y. <yan...@gm...> - 2014-05-22 13:57:21
|
By the way, I also made the same fix for vil1 test. While the disabling of line conversion on .pbm file fixed the tests on my Windows machine, it seems the two Windows machines on the dashboard still gave the same error: eastmodelsoft.com http://open.cdash.org/viewTest.php?onlyfailed&buildid=3338824 and NSK1200016973I.GE http://open.cdash.org/viewTest.php?onlyfailed&buildid=3339090 Could someone who have access to these two machines take a look into this issue? Regards, Gary Yang On Tue, May 20, 2014 at 6:16 PM, Joseph Mundy <mu...@le...> wrote: > Sean, > No. Keep the warnings. It should be the case that we eventually keep CLANG > happy. > Thanks, > Joe > > -----Original Message----- > From: Sean McBride [mailto:se...@ro...] > Sent: Tuesday, May 20, 2014 5:45 PM > To: Joseph Mundy; 'Gehua Yang'; 'Amitha Perera' > Cc: 'Vxl-maintainers' > Subject: Re: [Vxl-maintainers] test failure of vil/tests/vil_test_all > test_file_format_read > > On Tue, 20 May 2014 10:47:05 -0400, Joseph Mundy said: > >>We have been picking away at the brl errors. It seems CLANG is very picky. > > Joe, > > I think I'm the only one submitting any clang dashboards... there are many > warnings, yes, but it seems many of them are duplicated because the source > of the warning is a header included by many files. I suggest starting with > those, it should reduce the noise considerable. I can also disable warnings > that we feel are not useful or give too many false positives. > > Cheers, > > -- > ____________________________________________________________ > Sean McBride, B. Eng se...@ro... > Rogue Research www.rogue-research.com > Mac Software Developer Montréal, Québec, Canada > > |