<html>
<body>
Approved.<br><br>
<br>
<blockquote type=cite class=cite cite="">Change 20071108-hqm-4 by
hqm@IBM-2E06404CB67 on 2007-11-08 20:33:26 EST<br>
&nbsp;&nbsp;&nbsp; in /cygdrive/c/users/hqm/openlaszlo/trunk<br>
&nbsp;&nbsp;&nbsp; for
<a href="http://svn.openlaszlo.org/openlaszlo/trunk">
http://svn.openlaszlo.org/openlaszlo/trunk</a><br>
&nbsp;<br>
Summary: prevent spurious compiler warnings when overriding apply()
method on subclases of &lt;state&gt;<br>
&nbsp;<br>
New Features:<br>
&nbsp;<br>
Bugs Fixed: LPP-5067, LPP-5065<br>
&nbsp;<br>
Technical Reviewer: ptw<br>
QA Reviewer: pbr<br>
Doc Reviewer:<br>
&nbsp;<br>
Documentation:<br>
&nbsp;<br>
Release Notes:<br>
&nbsp;<br>
Details:<br>
&nbsp;<br>
&nbsp;<br>
I modified the compiler to not warn if the attribute being overriden was
declared explicitly with a value of &quot;true&quot; for it's 'override'
property.<br>
&nbsp;<br>
So in lfc.lzx, we declare apply as non overridable, on the base
Instance<br>
&nbsp;<br>
&lt;interface name=&quot;Instance&quot;
extends=&quot;Object&quot;&gt;<br>
&nbsp; &lt;method name=&quot;apply&quot;
override=&quot;false&quot;/&gt;<br>
&nbsp;<br>
but &quot;state&quot; declares it overridable, so any subclass of state
can override it<br>
&nbsp;<br>
&nbsp;<br>
&lt;interface name=&quot;state&quot; extends=&quot;node&quot;&gt;<br>
&nbsp; &lt;attribute name=&quot;apply&quot; override=&quot;true&quot;
/&gt;<br>
&nbsp;<br>
&nbsp;<br>
I made the compiler not only not warn if you override the method with
another method, but not warn if you override an attribute with a method.
That is a little too loose I guess, but it only happens when you declare
&quot;override=true&quot; on an attribute.<br>
&nbsp;<br>
&nbsp;<br>
Tests:<br>
&nbsp;<br>
Files:<br>
A&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;
test/compiler_errors/override-state-apply.lzx<br>
A&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;
test/compiler_errors/override-builtin.lzx<br>
M&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; WEB-INF/lps/schema/lfc.lzx<br>
M&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;
WEB-INF/lps/server/src/org/openlaszlo/compiler/ViewSchema.java<br>
M&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;
WEB-INF/lps/server/src/org/openlaszlo/compiler/NodeModel.java<br>
M&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;
WEB-INF/lps/misc/lzx-autoincludes.properties<br>
&nbsp;<br>
Changeset:
<a href="http://svn.openlaszlo.org/openlaszlo/patches/20071108-hqm-4.tar">
http://svn.openlaszlo.org/openlaszlo/patches/20071108-hqm-4.tar</a>
</blockquote></body>
</html>