review request for 6798511/6860431: Include functionality of Surrogate in Character
Xueming Shen
Xueming.Shen at Sun.COM
Thu Aug 27 18:03:38 UTC 2009
Ulf Zibis wrote:
> Am 26.08.2009 21:01, Xueming Shen schrieb:
>>
>> > - Fixed ugly output of make/tools/src/build.tools.spp.Spp; (see
>> jdk1.7.0/src.zip)
>>
>> Ulf, those buf.append(LNSEP) lines serve the purpose of keeping the
>> code in the
>> generated source file have exactly the same line number as they
>> appear in the
>> original source file.
>
> I have thought that before, but I _don't understand_ the *real value*
> against having "nice" code in externally delivered src.zip.
> Some time ago it was always annoying to me when debugger ran in that
> code, or I was just looking, how it works. (endless scrolling and
> often I oversaw the tail behind numbers of blank lines (I don't have
> line numbers on in my editor)).
> In contrast in generator output of SingleByte-X.java you don't care
> about line numbers, although it's in private sun package.
The original tool spp.sh has the goal of keeping the line numbers in
both the generated source file and the original -X.java file the
same, most of the time "we" read the -X.java version when debugging. My
re-writing follows that design/implementation.
In case of my SingleByte/DoubleByte-X, first those generated source
files are not exported to public, second, most coding logic
is in SingleByte/DoubleByte.java, the generated code is "purely the
holding bag of constant string tables, it's unlikely anyone will
run into debugging of those files. So I decided it's not worth keeping
the same like numbers in the output of SingleDoubleByte-X,
this is also one of the reasons that I keep them in the make/tools, not
in the src tree.
sherman
More information about the core-libs-dev
mailing list