review request for 6798511/6860431: Include functionality of Surrogate in Character

Ulf Zibis Ulf.Zibis at gmx.de
Thu Aug 27 19:55:39 UTC 2009


Am 27.08.2009 20:03, Xueming Shen schrieb:
> 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,

Have you seen my changes to SingleByte-X on 
https://bugs.openjdk.java.net/show_bug.cgi?id=100098 ? It's a step more 
to "purely" the holding bag of constant string tables.

> it's unlikely anyone will
> run into debugging of those files.

I am the guy who did this, but this is truly special case. :-)

> 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
>
>

Sherman, Kelly,

thanks for explaining me the *real value* .


-Ulf





More information about the core-libs-dev mailing list