Hi Roger,
On 2019-05-22 20:09, Roger Riggs wrote:
Hi Claes,
Properties.java: 542-547, 636-639 you might find the function Ivan added
useful:
newlen = ArraySupport.newLength(oldlength, min, preferred);
good suggestion - this is a slow path that we don't hit in any of the
startup tests I'm concerned with, so let's use that.
Can it be refactored differently to read the bytes into a char array and
then process that to avoid the processing code duplication.
.. not without a slowdown.
If not, can you put the two byte and char versions of readLine in
separate methods,
That massive if..else is ugly. Not sharing most of the code doing the
same things is quite ugly too.
... let's take a step back from that particular ugliness. :-)
http://cr.openjdk.java.net/~redestad/8224202/open.02/
- Joined stream/reader code paths back together
- Consume comment lines completely - reading new data from
stream/reader if we need to - removes the need for the
isCommentLine control variable
- Realized isNewLine is true iff len == 0, so it can be removed
This ends up being almost exactly as fast as open.01, with much
less code duplication. The break-to-label is unfortunate, of course.
/Claes
