RegexpWarnings in Goland

[Tommi2Day]

just to let you know Jetbrains Goland raises serveral RegExp warnings in range.go, which are causing force commits with changed code

Warning:(83, 49) Redundant character escape '\@' in RegExp
Warning:(83, 53) Unnecessary non-capturing group '(?:[-+]?[\d\.]+)'
Warning:(83, 64) Redundant character escape '\.' in RegExp
Warning:(83, 73) Unnecessary non-capturing group '(?:[-+]?[\d\.]+)'
Warning:(83, 84) Redundant character escape '\.' in RegExp
Warning:(83, 98) Unnecessary non-capturing group '(?:[-+]?[\d\.]+)'
Warning:(83, 109) Redundant character escape '\.' in RegExp
Warning:(83, 118) Unnecessary non-capturing group '(?:[-+]?[\d\.]+)'
Warning:(83, 129) Redundant character escape '\.' in RegExp
Warning:(84, 44) Unnecessary non-capturing group '(?:[-+]?[\d\.]+)'
Warning:(84, 55) Redundant character escape '\.' in RegExp
Warning:(84, 64) Unnecessary non-capturing group '(?:[-+]?[\d\.]+)'
Warning:(84, 75) Redundant character escape '\.' in RegExp
Warning:(85, 37) Unnecessary non-capturing group '(?:[-+]?[\d\.]+)'
Warning:(85, 48) Redundant character escape '\.' in RegExp
Warning:(85, 57) Unnecessary non-capturing group '(?:[-+]?[\d\.]+)'
Warning:(85, 68) Redundant character escape '\.' in RegExp

[atc0005]

@Tommi2Day Thanks for sharing this.

Does that IDE offer an option to automatically fix the regex patterns or an alternative?

[Tommi2Day]

partially yes. You can go through the positions complained and add "suggested Action"

[Tommi2Day]

this is a version with all suggestions adopted. Unit test ran sucessfully

digitOrInfinity := regexp.MustCompile([\d~]) optionalInvertAndRange := regexp.MustCompile(^@?([-+]?[\d.]+(?:e[-+]?[\d.]+)?|~)?(:([-+]?[\d.]+(?:e[-+]?[\d.]+)?)?)?$) firstHalfOfRange := regexp.MustCompile(^([-+]?[\d.]+(?:e[-+]?[\d.]+)?)?:) endOfRange := regexp.MustCompile(^[-+]?[\d.]+(?:e[-+]?[\d.]+)?$)

[atc0005]

@Tommi2Day Thanks for sharing. That linting feature looks like it would really come in handy.