> The #include line needs to be moved similarly.  I made that fix and a few 
> whitespace fixes.  I've attached a patch with those revisions.

Thanks.

> I noticed while making those fixes that you're using auto_ptr inside a map 
> value.  The auto_ptr is documented as not suitable for use in containers in 
> general.  For  TargetFolderBuildStreams you could just use a normal pointer 
> and then cleanup in the generator destructor using cmDeleteAll.

Good catch. I made the switch. Attached is a new patch. Here's the test: 
https://open.cdash.org/buildSummary.php?buildid=3741919.


Geoffrey Viola
SOFTWARE ENGINEER
asirobots.com

This message contains confidential information and is intended only for the 
recipient. If you are not the named addressee you should not disseminate, 
distribute or copy this e-mail. Please notify the sender immediately if you 
have received this e-mail by mistake and delete this e-mail from your system. 
Finally, the recipient should check this email and any attachments for the 
presence of viruses. The company accepts no liability for any damage caused by 
any virus transmitted by this email.

Attachment: 0001-Added-some-support-for-a-Green-Hills-MULTI.patch
Description: 0001-Added-some-support-for-a-Green-Hills-MULTI.patch

-- 

Powered by www.kitware.com

Please keep messages on-topic and check the CMake FAQ at: 
http://www.cmake.org/Wiki/CMake_FAQ

Kitware offers various services to support the CMake community. For more 
information on each offering, please visit:

CMake Support: http://cmake.org/cmake/help/support.html
CMake Consulting: http://cmake.org/cmake/help/consulting.html
CMake Training Courses: http://cmake.org/cmake/help/training.html

Visit other Kitware open-source projects at 
http://www.kitware.com/opensource/opensource.html

Follow this link to subscribe/unsubscribe:
http://public.kitware.com/mailman/listinfo/cmake-developers

Reply via email to