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