Approved!
Change 20080518-dda-6 by <mailto:[EMAIL PROTECTED]>[EMAIL PROTECTED]
on 2008-05-18 16:02:23 EDT
in /Users/dda/laszlo/src/svn/openlaszlo/trunk-c
for
<http://svn.openlaszlo.org/openlaszlo/trunk>http://svn.openlaszlo.org/openlaszlo/trunk
Summary: Script compiler line number rework
New Features:
Bugs Fixed: LPP-5784
Technical Reviewer: ptw (pending)
QA Reviewer: promanik (pending)
Doc Reviewer: (pending)
Documentation:
Release Notes:
Details:
General:
A fair amount of rework of the line numbers. We needed an integrated
approach so that internally at least, /* file: name#xx.yy */ wasn't
treated separately from /* file: #xx */. More line
information needed
to be preserved to capture all the various line transitions.
These
had various ripple effects, one being that versions of the LFC
(compiled with nameFunctions) needed to have trackLines turned off,
otherwise heap space blew up. This may be fixable in the
future if it
is important. LFC still has /* -*- file: .... */ comments for the
top of functions.
Also observed during testing is the necessity of not breaking output
lines to insert /* -*- file: .... */ comments. This breaks lzunit
test/lztest/lztest-node-options.js in a strange manner:
[exec] js: "test/lztest/lztest-node-options.js", line
108: uncaught JavaScript runtime exception: TypeError:
<mailto:[EMAIL PROTECTED]>[EMAIL PROTECTED]
is not a function, it is org.mozilla.javascript.Undefined.
A working lztest-node-options.js differed from the broken one only in
the comments produced. This remains a mystery, but without much
loss of line number accuracy, we now should not break lines.
lfc/build.xml
Turn off -tracklines when building the LFC, it can overflow
heap space
due to enormous strings generated internally.
ScriptCompiler.java, Compiler.java:
Eliminate needless call to Compiler.setProperties(),
and eliminate that method. Compiler.defaultOptions() is
now only called once.
Compiler.java:
Support disableTrackLines option.
Compiler.java, CommonGenerator.java:
Fragments generated with fake filenames are now issued with
[filename]
SWF9ParseTreePrinter.java:
More thorough checking of class 'type'
ParseTreePrinter.java
- Added extra debugging output for line numbers
(DEBUG_LINE_NUMBER)
- Integrate file/line numbering to a common approach.
There are still two annotations, but they both contain file/ line
numbers and they are processed the same. One is marked as a
'forced' line number so that a line number will sure to be
emitted at the beginning of a function.
- allow more internal line annotations to be created -
don't automatically elim an annotation just because it is
followed by another annotation. The intelligence about
what to finally produce happens at the end.
- added capability to extract a single annotation.
- encapsulate current and last line number info in
a LineNumberState struct to make comparison and updating
easier and clearer.
- when outputting the line numbers, fine tune the conditions for
when we need a new source location. Some of the factors in
the decision are:
-- is it the same as can be predicted?
-- does the current/previous filename have a name?
and is the name internal (used for compiling a fragment)?
and the line number >0?
-- are we at the beginning of a line (the preferred spot,
but there are exceptions)?
-- is this the beginning of a function?
-- is trackLines on?
this is all managed in shouldShowSourceLocation().
- there are now three variants of line number (source locations
using the nexb terminology) emitted:
/* -*- file: -*- */
There is no file attached to the following code
Don't expect to see line numbers updated.
/* -*- file: FILENAME#NN.YY -*- */
The filename has changed. The following output line
relates to line NN (and column YY) of FILENAME as the
input file. Subsequent output lines numbers can be
determined by simple incrementing.
/* -*- file: #NN */-*- */
The following output line relates to line NN of the
previously mentioned FILENAME as the input file.
Subsequent output lines numbers can be
determined by simple incrementing.
At the beginning of output, there is no file attached
until the first source location is seen. That is there is
an implicit /* -*- file: -*- */ at the beginning of output.
Tests:
Functionality:
Test cases from nexb #449 and #443 work by examining output JS.
In particular output not relating to particular input source lines
are indicated with /* -*- file: -*- */
<https://nexb.laszlosystems.com/trac/main/ticket/443>https://nexb.laszlosystems.com/trac/main/ticket/443
https://nexb.laszlosystems.com/trac/main/ticket/449
Compiled for example:
lzc -DnameFunctions --runtime=dhtml -S nexb443.lzx
Regression:
ant lzunit
swf9 hello
(smokecheck,weather,lzunit) x (swf8,dhtml)
Files:
M WEB-INF/lps/lfc/build.xml
M WEB-INF/lps/server/src/org/openlaszlo/sc/ScriptCompiler.java
M WEB-INF/lps/server/src/org/openlaszlo/sc/CommonGenerator.java
M WEB-INF/lps/server/src/org/openlaszlo/sc/
SWF9ParseTreePrinter.java
M WEB-INF/lps/server/src/org/openlaszlo/sc/Compiler.java
M WEB-INF/lps/server/src/org/openlaszlo/sc/ParseTreePrinter.java
Changeset:
<http://svn.openlaszlo.org/openlaszlo/patches/20080518-dda-6.tar>http://svn.openlaszlo.org/openlaszlo/patches/20080518-dda-6.tar
--
Don Anderson
Java/C/C++, Berkeley DB, systems consultant
voice: 617-547-7881
email: <mailto:[EMAIL PROTECTED]>[EMAIL PROTECTED]
www: <http://www.ddanderson.com>http://www.ddanderson.com
