2011/1/25 ryan weaver <[email protected]>:
> The @var are purposely left off of protected and private methods to
> declutter the code.

I think this shouldn't be a problem. Most IDEs collapse the docblocks
anyway if configured to do so.

> Now, at the very least, I see from your example that classes that are
> commonly subclassed - like ContainerAware - should probably have the @var.
> I'd recommend making a pull request with that change and state your case - I
> think you have a point.

I agree. @Daniel: Can you add the @var-comments where you find them
missing and send a pull request? This would help us a lot.

Cheers,
Bernhard
--
Software Architect & Engineer
Blog: http://webmozarts.com
Twitter: http://twitter.com/webmozart

-- 
If you want to report a vulnerability issue on symfony, please send it to 
security at symfony-project.com

You received this message because you are subscribed to the Google
Groups "symfony developers" group.
To post to this group, send email to [email protected]
To unsubscribe from this group, send email to
[email protected]
For more options, visit this group at
http://groups.google.com/group/symfony-devs?hl=en

Reply via email to