Maven Doxia

Decouple end-of-line detection in LineBreaker from OS

Details

  • Type: Improvement Improvement
  • Status: Closed Closed
  • Priority: Trivial Trivial
  • Resolution: Fixed
  • Affects Version/s: 1.1
  • Fix Version/s: 1.1
  • Component/s: Core
  • Labels:
    None
  • Patch Submitted:
    Yes
  • Number of attachments :
    1

Description

To my knowledge, all proper text formats accepts "\r\n" as a single line terminator upon input, regardless of the operating system a parser is run on. The difference about line terminators usually only manifests itself when it comes to writing text. Hence, LineBreaker should not try to detect a Win box but check for the char sequence "\r\n".

If you insist on OS-based behaviour, Os.isFamily( Os.FAMILY_WINDOWS ) from plexus-utils should be used and its return value should be cached in an instance variable of LineBreaker instead of querying the system property "os.name" for every single character.

Activity

Hide
Lukas Theussl added a comment -

Fixed in r628850. Thanks!

Show
Lukas Theussl added a comment - Fixed in r628850. Thanks!

People

Vote (0)
Watch (0)

Dates

  • Created:
    Updated:
    Resolved: