# Type hinting/casting request for vote - Thread: https://externals.io/message/44727 - Original archive: https://news-web.php.net/php.internals/44727 - Started by Ilia Alshanetsky on 2009-07-07T00:52:19+00:00 - Messages: 93 - Page 1 of 4: messages 1 to 25 (25 per page, add `?page=N` to the URL) - Next page: https://externals.io/message/44727.md?page=2 - Reply by email to internals@lists.php.net - More for agents: search with https://externals.io/search.md?q={query} · JSON API at https://externals.io/api · documentation at https://externals.io/llms.txt --- ## #44727 — Ilia Alshanetsky — 2009-07-07T00:52:19+00:00 URL: https://externals.io/message/44727#44727 Last week or so there was a fairly detailed discussion on the internals list regarding type hinting based on my original patch. Since then the patch has been revised to address the major concerns that were identified (breakage of binary compatibility) as well extended with additional functionality (object hint, type casting, reflection support, etc...). The final patch is available here: http://ilia.ws/patch/type_hint_53_v2.txt The test suit is available here: http://ia.gd/patch/type_hint_tests.tar.bz2 I would like to ask all developers to voice their opinions of whether it makes sense to add this to 5.3 or to throw it away (either one is fine btw). To keep the process simple & flamewar free, please restrict yourself to +/- (1/0), next week monday I'll run a tally of the votes and based on the result we can determine how to proceed further. Ilia --- ## #44728 — Felipe Pena — 2009-07-07T01:40:26+00:00 URL: https://externals.io/message/44727#44728 · In reply to: #44727 2009/7/6 Ilia Alshanetsky : > I would like to ask all developers to voice their opinions of whether it > makes sense to add this to 5.3 or to throw it away (either one is fine btw). > To keep the process simple & flamewar free, please restrict yourself to +/- > (1/0), next week monday I'll run a tally of the votes and based on the > result we can determine how to proceed further. +1 -- Regards, Felipe Pena --- ## #44729 — Eddie Drapkin — 2009-07-07T01:42:13+00:00 URL: https://externals.io/message/44727#44729 · In reply to: #44727 > Last week or so there was a fairly detailed discussion on the internals list > regarding type hinting based on my original patch. Since then the patch has > been revised to address the major concerns that were identified (breakage of > binary compatibility) as well extended with additional functionality (object > hint, type casting, reflection support, etc...). > > The final patch is available here: http://ilia.ws/patch/type_hint_53_v2.txt > The test suit is available here: http://ia.gd/patch/type_hint_tests.tar.bz2 > > I would like to ask all developers to voice their opinions of whether it > makes sense to add this to 5.3 or to throw it away (either one is fine btw). > To keep the process simple & flamewar free, please restrict yourself to +/- > (1/0), next week monday I'll run a tally of the votes and based on the > result we can determine how to proceed further. > > Ilia > > -- > > > > +1 --- ## #44730 — Guilherme Blanco — 2009-07-07T02:00:04+00:00 URL: https://externals.io/message/44727#44730 · In reply to: #44729 +1 > > > […] > > > +1 > > -- > > > > -- Guilherme Blanco - Web Developer CBC - Certified Bindows Consultant Cell Phone: +55 (16) 9215-8480 MSN: guilhermeblanco@hotmail.com URL: http://blog.bisna.com São Paulo - SP/Brazil --- ## #44731 — Adam Ashley — 2009-07-07T03:00:25+00:00 URL: https://externals.io/message/44727#44731 · In reply to: #44727 > I would like to ask all developers to voice their opinions of whether it > makes sense to add this to 5.3 or to throw it away (either one is fine btw). > To keep the process simple & flamewar free, please restrict yourself to +/- > (1/0), next week monday I'll run a tally of the votes and based on the > result we can determine how to proceed further. +1 --- ## #44733 — Alexey Zakhlestin — 2009-07-07T05:49:19+00:00 URL: https://externals.io/message/44727#44733 · In reply to: #44727 > Last week or so there was a fairly detailed discussion on the internals list > regarding type hinting based on my original patch. Since then the patch has > been revised to address the major concerns that were identified (breakage of > binary compatibility) as well extended with additional functionality (object > hint, type casting, reflection support, etc...). > > The final patch is available here: http://ilia.ws/patch/type_hint_53_v2.txt > The test suit is available here: http://ia.gd/patch/type_hint_tests.tar.bz2 > > I would like to ask all developers to voice their opinions of whether it > makes sense to add this to 5.3 or to throw it away (either one is fine btw). > To keep the process simple & flamewar free, please restrict yourself to +/- > (1/0), next week monday I'll run a tally of the votes and based on the > result we can determine how to proceed further. +1 -- Alexey Zakhlestin http://www.milkfarmsoft.com/ --- ## #44735 — Matthew Fonda — 2009-07-07T06:43:49+00:00 URL: https://externals.io/message/44727#44735 · In reply to: #44727 > Last week or so there was a fairly detailed discussion on the internals list regarding type hinting based on my original patch. Since then the patch has been revised to address the major concerns that were identified (breakage of binary compatibility) as well extended with additional functionality (object hint, type casting, reflection support, etc...). +1 Regards, Matthew > > The final patch is available here: http://ilia.ws/patch/type_hint_53_v2.txt > The test suit is available here: http://ia.gd/patch/type_hint_tests.tar.bz2 > > I would like to ask all developers to voice their opinions of whether it makes sense to add this to 5.3 or to throw it away (either one is fine btw). To keep the process simple & flamewar free, please restrict yourself to +/- (1/0), next week monday I'll run a tally of the votes and based on the result we can determine how to proceed further. > > Ilia --- ## #44736 — Sebastian Bergmann — 2009-07-07T06:44:48+00:00 URL: https://externals.io/message/44727#44736 · In reply to: #44727 Ilia Alshanetsky schrieb: > I would like to ask all developers to voice their opinions of whether it > makes sense to add this to 5.3 or to throw it away (either one is fine > btw). To keep the process simple & flamewar free, please restrict > yourself to +/- (1/0), next week monday I'll run a tally of the votes > and based on the result we can determine how to proceed further. I have tested the patch (ran the supplied test suite, reviewed the test suite) and ran into no issues. +1 -- Sebastian Bergmann Co-Founder and Principal Consultant http://sebastian-bergmann.de/ http://thePHP.cc/ --- ## #44737 — Derick Rethans — 2009-07-07T07:34:30+00:00 URL: https://externals.io/message/44727#44737 · In reply to: #44727 > Last week or so there was a fairly detailed discussion on the internals list > regarding type hinting based on my original patch. Since then the patch has > been revised to address the major concerns that were identified (breakage of > binary compatibility) as well extended with additional functionality (object > hint, type casting, reflection support, etc...). > > The final patch is available here: http://ilia.ws/patch/type_hint_53_v2.txt > The test suit is available here: http://ia.gd/patch/type_hint_tests.tar.bz2 > > I would like to ask all developers to voice their opinions of whether it makes > sense to add this to 5.3 or to throw it away (either one is fine btw). To keep > the process simple & flamewar free, please restrict yourself to +/- (1/0), > next week monday I'll run a tally of the votes and based on the result we can > determine how to proceed further. +1 Derick -- http://derickrethans.nl | http://ezcomponents.org | http://xdebug.org twitter: @derickr --- ## #44739 — Mac tus — 2009-07-07T08:22:42+00:00 URL: https://externals.io/message/44727#44739 · In reply to: #44737 +1 --- ## #44740 — konference@kukulich.net — 2009-07-07T08:28:11+00:00 URL: https://externals.io/message/44727#44740 · In reply to: #44727 +1 --- ## #44741 — Michael Maclean — 2009-07-07T08:34:00+00:00 URL: https://externals.io/message/44727#44741 · In reply to: #44727 Ilia Alshanetsky wrote: > I would like to ask all developers to voice their opinions of whether it > makes sense to add this to 5.3 or to throw it away (either one is fine > btw). To keep the process simple & flamewar free, please restrict > yourself to +/- (1/0), next week monday I'll run a tally of the votes > and based on the result we can determine how to proceed further. +1 --- ## #44742 — Hannes Magnusson — 2009-07-07T08:52:13+00:00 URL: https://externals.io/message/44727#44742 · In reply to: #44727 > The final patch is available here: http://ilia.ws/patch/type_hint_53_v2.txt > The test suit is available here: http://ia.gd/patch/type_hint_tests.tar.bz2 +1 -Hannes --- ## #44743 — Pierre Joye — 2009-07-07T09:42:22+00:00 URL: https://externals.io/message/44727#44743 · In reply to: #44727 +1 -- Pierre http://blog.thepimp.net | http://www.libgd.org --- ## #44747 — Patrick ALLAERT — 2009-07-07T10:57:56+00:00 URL: https://externals.io/message/44727#44747 · In reply to: #44727 +1 Patrick Allaert --- http://code.google.com/p/peclapm/ - Alternative PHP Monitor --- ## #44748 — David Coallier — 2009-07-07T11:29:52+00:00 URL: https://externals.io/message/44727#44748 · In reply to: #44747 +1 -- Slan, David --- ## #44750 — crrodriguez@suse.de — 2009-07-07T12:40:07+00:00 URL: https://externals.io/message/44727#44750 · In reply to: #44727 > The final patch is available here: http://ilia.ws/patch/type_hint_53_v2.txt > The test suit is available here: http://ia.gd/patch/type_hint_tests.tar.bz2 +1 --- ## #44751 — Richard Quadling — 2009-07-07T12:44:55+00:00 URL: https://externals.io/message/44727#44751 · In reply to: #44727 2009/7/7 Ilia Alshanetsky : > Last week or so there was a fairly detailed discussion on the internals list > regarding type hinting based on my original patch. Since then the patch has > been revised to address the major concerns that were identified (breakage of > binary compatibility) as well extended with additional functionality (object > hint, type casting, reflection support, etc...). > > The final patch is available here: http://ilia.ws/patch/type_hint_53_v2.txt > The test suit is available here: http://ia.gd/patch/type_hint_tests.tar.bz2 > > I would like to ask all developers to voice their opinions of whether it > makes sense to add this to 5.3 or to throw it away (either one is fine btw). > To keep the process simple & flamewar free, please restrict yourself to +/- > (1/0), next week monday I'll run a tally of the votes and based on the > result we can determine how to proceed further. > > Ilia > > -- > > > > +1 -- ----- Richard Quadling Zend Certified Engineer : http://zend.com/zce.php?c=ZEND002498&r=213474731 "Standing on the shoulders of some very clever giants!" I need a car : http://snipurl.com/l4pih ZOPA : http://uk.zopa.com/member/RQuadling --- ## #44752 — Rafael Dohms — 2009-07-07T13:29:00+00:00 URL: https://externals.io/message/44727#44752 · In reply to: #44751 +1 Rafael Dohms On Tue, Jul 7, 2009 at 9:44 AM, Richard Quadling wrote: > 2009/7/7 Ilia Alshanetsky : > > […] > list > > […] > has > > […] > of > > […] > (object > > […] > http://ilia.ws/patch/type_hint_53_v2.txt > > […] > http://ia.gd/patch/type_hint_tests.tar.bz2 > > […] > btw). > > […] > +/- > > […] > > +1 > > > -- > ----- > Richard Quadling > Zend Certified Engineer : http://zend.com/zce.php?c=ZEND002498&r=213474731 > "Standing on the shoulders of some very clever giants!" > I need a car : http://snipurl.com/l4pih > ZOPA : http://uk.zopa.com/member/RQuadling --- ## #44754 — Greg Beaver — 2009-07-07T13:58:00+00:00 URL: https://externals.io/message/44727#44754 · In reply to: #44727 Ilia Alshanetsky wrote: > Last week or so there was a fairly detailed discussion on the internals > list regarding type hinting based on my original patch. Since then the > patch has been revised to address the major concerns that were > identified (breakage of binary compatibility) as well extended with > additional functionality (object hint, type casting, reflection support, > etc...). > > The final patch is available here: http://ilia.ws/patch/type_hint_53_v2.txt > The test suit is available here: http://ia.gd/patch/type_hint_tests.tar.bz2 > > I would like to ask all developers to voice their opinions of whether it > makes sense to add this to 5.3 or to throw it away (either one is fine > btw). To keep the process simple & flamewar free, please restrict > yourself to +/- (1/0), next week monday I'll run a tally of the votes > and based on the result we can determine how to proceed further. +1 if the object type hint is change to use `T_CLASS` so we don't break every external package using "Object" as a base class. http://www.google.com/codesearch?as_q=class\s%2Bobject&btnG=Search+Code&hl=en&as_lang=php&as_license_restrict=i&as_license=&as_package=&as_filename=&as_case= Greg --- ## #44755 — James Dempster — 2009-07-07T14:05:33+00:00 URL: https://externals.io/message/44727#44755 · In reply to: #44727 +1 from me > Last week or so there was a fairly detailed discussion on the internals > list regarding type hinting based on my original patch. Since then the patch > has been revised to address the major concerns that were identified > (breakage of binary compatibility) as well extended with additional > functionality (object hint, type casting, reflection support, etc...). > > The final patch is available here: > http://ilia.ws/patch/type_hint_53_v2.txt > The test suit is available here: > http://ia.gd/patch/type_hint_tests.tar.bz2 > > I would like to ask all developers to voice their opinions of whether it > makes sense to add this to 5.3 or to throw it away (either one is fine btw). > To keep the process simple & flamewar free, please restrict yourself to +/- > (1/0), next week monday I'll run a tally of the votes and based on the > result we can determine how to proceed further. > > Ilia --- ## #44756 — Christian Schneider — 2009-07-07T14:08:08+00:00 URL: https://externals.io/message/44727#44756 · In reply to: #44754 Greg Beaver wrote: > > […] > > +1 if the object type hint is change to use `T_CLASS` so we don't break > every external package using "Object" as a base class. -1 because of the new keywords bool, boolean, string, binary, unicode, int, integer, long, real, double, float, resource, numeric, scalar, object which cannot be used as function or class names any more (major BC break). I have more comments but they are minor so I restrict myself to this big one... Cheers, - Chris --- ## #44757 — Mark van der Velden — 2009-07-07T14:10:47+00:00 URL: https://externals.io/message/44727#44757 · In reply to: #44727 Ilia Alshanetsky wrote: > [..] > I would like to ask all developers to voice their opinions of whether it > makes sense to add this to 5.3 or to throw it away (either one is fine > btw). +1 > [..] - Mark --- ## #44758 — Elizabeth M Smith — 2009-07-07T14:14:40+00:00 URL: https://externals.io/message/44727#44758 · In reply to: #44727 Ilia Alshanetsky wrote: > Last week or so there was a fairly detailed discussion on the internals > list regarding type hinting based on my original patch. Since then the > patch has been revised to address the major concerns that were > identified (breakage of binary compatibility) as well extended with > additional functionality (object hint, type casting, reflection support, > etc...). > > The final patch is available here: http://ilia.ws/patch/type_hint_53_v2.txt > The test suit is available here: http://ia.gd/patch/type_hint_tests.tar.bz2 > > I would like to ask all developers to voice their opinions of whether it > makes sense to add this to 5.3 or to throw it away (either one is fine > btw). To keep the process simple & flamewar free, please restrict > yourself to +/- (1/0), next week monday I'll run a tally of the votes > and based on the result we can determine how to proceed further. > > Ilia +1 with the same caveat as Greg (please object not Object) Thanks, Elizabeth --- ## #44759 — Stan Vassilev — 2009-07-07T14:15:54+00:00 URL: https://externals.io/message/44727#44759 · In reply to: #44754 > +1 if the object type hint is change to use `T_CLASS` so we don't break > every external package using "Object" as a base class. > > http://www.google.com/codesearch?as_q=class\s%2Bobject&btnG=Search+Code&hl=en&as_lang=php&as_license_restrict=i&as_license=&as_package=&as_filename=&as_case= > > Greg Or this can wait until 6.0, when (as I hear) we'll have case-sensitive class names, so Object/object, Int/int won't cause collisions. I'm really puzzled why a non-essential, and for the past months (years?) controversial and always rejected feature such as strict type hints, has everyone turning 180 degrees and putting it in a minor release in the course of a week. Regards, Stan Vassilev --- Next page: https://externals.io/message/44727.md?page=2