[mhahn]

I would argue that it is more readable to do:

if request.pipeline.command_stack: request.pipeline.execute()

rather than

if len(request.pipeline) > 0: request.pipeline.execute()

thoughts?

[raverbashing]

The 1st example is better

(Yes, if len returns 0 it's evaluated as false, just tested this)

But for the 2nd I would prefer

if not len(request.pipeline):

[mhahn]

if len(request.pipeline):

you want to execute only if it has a length.

cool though, I am using pipelines for something at work and read this post and realized I was executing even if there was nothing to execute!

[raverbashing]

Oh, of course =)

I don't know why I put the 'not' there