Feature #9881

Improve the error message of Squiz.ControlStructures.ControlSignature

Added by Stefano Kowalke over 2 years ago. Updated 19 days ago.

Status:Accepted Start date:2010-09-22
Priority:Should have Due date:
Assignee:- % Done:

0%
Category:ControlStructures Spent time: -
Target version:0.0.7
Branch: Tags:ControlStructures
Votes: 0

Description

The error message was not clear enough:

Expected "if (...) {\n"; found "if (...) {\n"

Expected and found lines are the same, but means different things. The code in questions have a tab plus space between the closing brace and the opening curly brace, but it wasn't reflect by the sniff.

if (...)[tab][space]{\n
if (...)[tab]{\n

Better error message could be:

Expected "if (...)[space]{\n"; found "if (...)[tab]{\n

History

#1
Updated by Stefano Kowalke over 2 years ago

  • Status changed from New to Accepted
  • Target version changed from Initial release to 0.0.5

#2
Updated by Stefano Kowalke about 1 year ago

  • Tags set to ControlStructures

#3
Updated by Stefano Kowalke 8 months ago

  • Target version changed from 0.0.5 to 0.0.6

#4
Updated by Stefano Kowalke 19 days ago

  • Target version changed from 0.0.6 to 0.0.7

Also available in: Atom PDF