[TYPO3-core] CGL Voting for typecasts and @var annotations

Aske Ertmann aske at moc.net
Fri Jan 17 13:31:35 CET 2014


Hey

As Helmut says it’s “@var integer” not “@var Integer”. It has nothing to do with classes. int is just a shorthand for integer, but they’re exactly the same.

http://www.php.net/manual/en/language.types.integer.php
http://www.php.net/manual/en/language.types.type-juggling.php

On 17/01/2014, at 13.09, Markus Klein <klein.t3 at mfc-linz.at> wrote:

> @var Integer.... hm
> 
> 1.) Integer is the class not the basic data type
> 2.) Isn't there a \ missing in front, for global namespace?
> 3.) Is there any benefit of using the classes instead of the basic data types?
> 
> Kind regards
> Markus
> 
> ------------------------------------------------------------
> Markus Klein
> TYPO3 CMS Active Contributors Team Member
> 
> From: typo3-team-core-bounces at lists.typo3.org [mailto:typo3-team-core-bounces at lists.typo3.org] On Behalf Of Fabien Udriot
> Sent: Friday, January 17, 2014 1:02 PM
> To: TYPO3 core team
> Subject: Re: [TYPO3-core] CGL Voting for typecasts and @var annotations
> 
> Hi,
> It would be good to have the same convention across the TYPO3 project. 
> Isn't it used "@var Integer" in Flow?
> Fb.
> Hi!
> On 16.01.14 11:11, Dmitry Dulepov wrote:
> Hi!
> Helmut Hummel wrote:
> Since this topic pops up from time to time when speaking with different
> people, I'd like to have a final and official decision on it.
> On what exactly? Could you be more specific?
> 1.
> How typecasts should look like. Long name (integer) or short name (bool), with or without space before the variable.
> 2.
> How @var annotations look like. Long name "@var integer" or short name "@var int" and whether the decision on this should depend on the decision on typecasts or if this can be different (typecast short, var annotation long).
> Kind regards,
> Helmut
> -- 
> Helmut Hummel
> Release Manager TYPO3 6.0
> TYPO3 Core Developer, TYPO3 Security Team Member
> TYPO3 .... inspiring people to share!
> Get involved: typo3.org
> ________________________________________
> Before posting to this list, please have a look to the posting rules
> on the following websites:
> http://typo3.org/teams/core/core-mailinglist-rules/
> http://typo3.org/development/bug-fixing/diff-and-patch/
> ________________________________________
> TYPO3-team-core mailing list
> TYPO3-team-core at lists.typo3.org
> http://lists.typo3.org/cgi-bin/mailman/listinfo/typo3-team-core
> 
> _______________________________________________
> Before posting to this list, please have a look to the posting rules
> on the following websites:
> 
> http://typo3.org/teams/core/core-mailinglist-rules/
> http://typo3.org/development/bug-fixing/diff-and-patch/
> _______________________________________________
> TYPO3-team-core mailing list
> TYPO3-team-core at lists.typo3.org
> http://lists.typo3.org/cgi-bin/mailman/listinfo/typo3-team-core

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.typo3.org/pipermail/typo3-team-core/attachments/20140117/361b12c2/attachment.htm>


More information about the TYPO3-team-core mailing list