On 12/19/2012 07:31 AM, Stephen Kelly wrote: > I think this idea gets complex quickly.
Yes, we're bikeshedding it. > The cmPropertyMap would need to hold cmProperty* and new them. cmPropertyMap > and cmProperties are currently copied but that would have to be made > impossible as it introduces ownership problems. > > As we only currently have one potential user of this feature (the > INCLUDE_DIRECTORIES), I'd prefer to stick with a solution which does not > require making the cmProperty API virtual. I also see benefits for making it > virtual, but I don't have a grasp on how much work other work it creates. Okay. Since this is purely internal it can be factored into a generalized approach later. -Brad -- 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
