patchClasspath Extensions - Patches: patch #8294, gnu.inet.util.CRLFOutputStream add...

 
 

You are not allowed to post comments on this tracker with your current authentification level.

patch #8294: gnu.inet.util.CRLFOutputStream add beginning of line field: atBOL

Submitted by:  Conrad T. Pino <conradpino>
Submitted on:  Tue Jan 28 19:15:54 2014  
 
Category: NonePriority: 5 - Normal
Status: DonePrivacy: Public
Assigned to: Conrad T. Pino <conradpino>Open/Closed: Open

Tue Jan 28 23:20:41 2014, comment #3:

svn ci -m "patch #8294: gnu.inet.util.CRLFOutputStream add beginning of line field: atBOL" trunk/inetlib

Committed revision 1239.

Conrad T. Pino <conradpino>
Project MemberIn charge of this item.
Tue Jan 28 23:10:57 2014, comment #2:

Prior patch was incomplete; it failed to reset atBOL to false when method "public void write(byte[] b, int off, int len)" buffer did not end with newline sequence.

Issues second patch addresses are:

1. Remove white space between method name and "(" character.

2. Optimize duplicated "len - d" with "len -= d" and "len".

3. Correct atBOL reset omission.

(file #30399)

Conrad T. Pino <conradpino>
Project MemberIn charge of this item.
Tue Jan 28 19:44:13 2014, comment #1:

svn ci -m "patch #8294: gnu.inet.util.CRLFOutputStream add beginning of line field: atBOL" trunk/inetlib

Committed revision 1237.

Conrad T. Pino <conradpino>
Project MemberIn charge of this item.
Tue Jan 28 19:15:54 2014, original submission:

1. Add "protected boolean atBOL;" field.

2. Replace ASCII decimal (13, 10) with character ('\r', '\n') constants.

3. Replace if else if else ladder with switch statement.

4. Replace all "out.write(CRLF...)" with "writeln()" statement.

5. Replace decimal constant "2" with "CRLF.length" value.

Conrad T. Pino <conradpino>
Project MemberIn charge of this item.

 

Attached Files
file #30399:  CRLFOutputStream.patch added by conradpino (977B - application/octet-stream - Supplemental patch; prior file required as well.)
file #30396:  CRLFOutputStream.patch added by conradpino (2kB - application/octet-stream)

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -unavailable- added by conradpino (Submitted the item)
  • -unavailable- added by conradpino
  •  

    Do you think this task is very important?
    If so, you can click here to add your encouragement to it.
    This task has 0 encouragements so far.

    Only logged-in users can vote.

     

    Please enter the title of George Orwell's famous dystopian book (it's a date):

     

     

    Follow 6 latest changes.

    Date Changed By Updated Field Previous Value => Replaced By
    Tue Jan 28 23:20:41 2014conradpinoStatusIn Progress=>Done
    Tue Jan 28 23:10:57 2014conradpinoAttached File-=>Added CRLFOutputStream.patch, #30399
      StatusDone=>In Progress
    Tue Jan 28 19:44:13 2014conradpinoStatusIn Progress=>Done
    Tue Jan 28 19:15:54 2014conradpinoAttached File-=>Added CRLFOutputStream.patch, #30396
      Carbon-Copy-=>Added dog

    Back to the top


    Powered by Savane 3.1-cleanup