[svn.haxx.se] · SVN Dev · SVN Users · SVN Org · TSVN Dev · TSVN Users · Subclipse Dev · Subclipse Users · this month's index

Re: svn commit: r1487954 - /subversion/trunk/build/generator/gen_base.py

From: Branko Čibej <brane_at_apache.org>
Date: Fri, 31 May 2013 14:56:17 +0200

On 31.05.2013 14:52, Julian Foad wrote:
> Bert Huijben wrote:
>
>>> Author: julianfoad
>>> URL: http://svn.apache.org/r1487954
>>> Log:
>>> * build/generator/gen_base.py
>>> (TargetLinked): Remove some redundant and wrong code.
>>> (_collect_paths): Document what happens if the pattern is not found.
>>>
>>> Modified: subversion/trunk/build/generator/gen_base.py
>>> ==========================================================
>>> @@ -471,10 +471,9 @@ class TargetLinked(Target):
>>> # the specified install area depends upon this target
>>> self.gen_obj.graph.add(DT_INSTALL, self.install, self)
>>>
>>> - sources = sorted(_collect_paths(self.sources or '*.c' or '*.cpp', self.path))
>>> + sources = sorted(_collect_paths(self.sources, self.path))
>> I don't understand the use of the "or '*.cpp'" part as
>> that code appears to dead,
> Exactly: the "or '*.cpp'" would never be executed, so it was a mistake by whoever wrote it.

Might have been me ...

-- Brane
Received on 2013-05-31 14:56:25 CEST

This is an archived mail posted to the Subversion Dev mailing list.

This site is subject to the Apache Privacy Policy and the Apache Public Forum Archive Policy.