Re: [Vxl-users] Mac OS X dashboards & adding clang support

[Sean M. <se...@ro...>]

On Thu, 29 Jul 2010 20:21:06 +0000, Peter Vanroose said:

>> Then I tried another build with all the same settings
>> except using clang and there are hundreds of errors:
>> <http://www.cdash.org/CDash/viewBuildError.php?buildid=660226>
>> 
>> I'm not a C++ expert, but many of the errors look legitimate.
>> 
>> Anyone interested in helping getting this building?
>
>I've looked at these errors, and 
>
> *SNIP*
>
>From yesterday's dashboard of the clang compiler, it seems like these
>modifications will help to reduce the number of errors. (And the other
>builds look very green today -- seems nothing got broken.)
>To be continued...

Peter,

Thanks a lot for looking at this and fixing some issues!  It's indeed
much better now, instead of reaching the dashboard max of 500, there are
now "only" 354 errors:

<http://www.cdash.org/CDash/viewBuildError.php?buildid=680234>

That may also be due to the fact that I updated my build of clang.  They
are fast improving their C++ support.

One error is repeated often and looks simple enough to fix:

/.../vxl/contrib/prip/vmap/vmap_ptr_sequences.h:295:5: error: use of
undeclared identifier 'set_begin'
     set_begin(new pointer[arg_size]) ;
     ^
     this->

It appears to be saying "this->" must be added.

Likely some of the errors are bugs in clang.  Just this week several of
the VTK developers got VTK building with clang... There were two issues
that were clang bugs, and the clang team fixed them each within a day,
and now VTK builds!

Cheers,

-- 
____________________________________________________________
Sean McBride, B. Eng                 se...@ro...
Rogue Research                        www.rogue-research.com 
Mac Software Developer              Montréal, Québec, Canada