[Laszlo-dev] For Review: Change 20080403-bargull-0 Summary: change LzDataset default-name to new regime

[André Bargull]

"operatorArgs" can be either 'null' or '0' and there is the following 
test in "LzDatapointer#__LZprocessOperator(..)":
> if ( pp.operatorArgs != null ){
>     return p[ pp.operator ] ( pp.operatorArgs );
> }
But if AS3 does not support "nullable Number", I'll just comment that 
type-info out resp. change it to "var operatorArgs:*" .


P T Withington wrote:
> Yeah, probably just not (yet) supported in AS3.  But do you really 
> want a nullable number?  Is there not a better default, like maybe 0?
>
> On 2008-04-03, at 14:29 EDT, André Bargull wrote:
>> I thought you could have nullable Numbers if you prepend '?' to the 
>> type, at least I found that somewhere in the ES4 papers. And our 
>> compiler does accept that syntax.
>> So, it's just not allowed in AS3?
>>
>> - André
>>
>>
>> On 4/3/2008 8:12 PM, Philip Romanik wrote:
>>> Hi André,
>>>
>>> Approved, but you need to make one change in LzParsedPath:
>>>
>>>        var operatorArgs:Number? = null;
>>>
>>> This should be:
>>>
>>>        var operatorArgs = null;
>>>
>>> A number can't be null, and operatorArgs can be null.
>>>
>>>
>>> Also, there is no Jira bug for this changeset. There must be a Jira 
>>> bug to file against.
>>>
>>> Thanks!
>>>
>>> Phil
>>>
>>>
>>>
>>>
>>>
>>>> Change 20080403-bargull-0 by bargull at dell--p4--2-53 on 2008-04-03 
>>>> 17:49:17 in /home/Admin/src/svn/openlaszlo/trunk
>>>> for http://svn.openlaszlo.org/openlaszlo/trunk
>>>> Summary: change LzDataset default-name to new regime
>>>> New Features:
>>>> Bugs Fixed:
>>>> Technical Reviewer: promanik
>>>> QA Reviewer: ptw
>>>> Doc Reviewer: (pending)
>>>> Documentation:
>>>> Release Notes:
>>>> Details:
>>>> - change default-name for LzDataset to new regime
>>>> - update AnonDatasetGenerator to use new regime
>>>> - add more type-information to LzParsedPath
>>>>  Tests:
>>>> alldata.lzx still works
>>>> Files:
>>>> M WEB-INF/lps/lfc/data/LzParsedPath.lzs
>>>> M WEB-INF/lps/lfc/data/LzDataset.lzs
>>>> Changeset:
>>>> http://svn.openlaszlo.org/openlaszlo/patches/20080403-bargull-0.tar
>>>>
>
>