On Wednesday 20 February 2013, Stephen Kelly wrote:
> Brad King wrote:
> > Please fix and add this case to the tests.
> 
> I've added fix-automoc-linker-language to stage. Alex, could you review
> please?

If I see it correctly, actually nothing is done to each target between the 
calls to InitializeMocSourceFile() and the following SetupAutomocTarget().
So instead of running two loops over the targets, shouldn't it be enough to 
add the source file to the target early on in SetupAutomocTarget(), e.g. 
before the call to AddUtilityCommand() ?

Alex
--

Powered by www.kitware.com

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

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

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

Reply via email to