Skip to content

PEAR.ControlStructures.MultiLineCondition doesn't account for comments at the end of lines #407

@aik099

Description

@aik099

Original code:

// Check the last error code now... if ($IPP->errorCode() == 401 or // most calls return this. $IPP->errorCode() == 3200) // but for some stupid reason the getAvailableCompanies call returns this. { return false; }

Fixed code (wrong):

// Check the last error code now... if ($IPP->errorCode() == 401 or // most calls return this. $IPP->errorCode() == 3200 ) // but for some stupid reason the getAvailableCompanies call returns this. { return false; }

Fixed code (expected):

// Check the last error code now... if ($IPP->errorCode() == 401 // most calls return this. or $IPP->errorCode() == 3200 // but for some stupid reason the getAvailableCompanies call returns this. ) { return false; }

2 changes that were not made:

  • or wasn't moved to the beginning of next line
  • ) { wasn't placed on new line

Metadata

Metadata

Assignees

No one assigned

    Labels

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions