Code review comment for lp:~salgado/summit/highlight-linaro-sessions

Revision history for this message
Guilherme Salgado (salgado) wrote :

On Wed, 2010-09-22 at 15:41 +0000, Michael Hall wrote:
> In def is_linaro, is there a reason you didn't just include the leading / in the string literal, thus saving the call to startswith() and a string concatination?

Just to avoid repeating the slash in all lines, I guess. Note that the
startswith() call can't be avoided, though, as the path won't ever be
just /project for a blueprint URL.

--
Guilherme Salgado <https://launchpad.net/~salgado>

« Back to merge proposal