[RFE] linter: option to relax rule requiring braces for single statements

[chrizilla]

summary

Firemonkey linter currently issues warnings about missing braces for all single-statement blocks (e.g. if (cond) return; must be if (cond) { return; }).

steps to reproduce

1. create a userscript with single-line blocks:

   if (a !== b) return false;

⛔ actual behaviour

Firemonkey linter complains about missing curly braces:
⚠️ Expected "{" and instead saw "return"

✅ expected behavior

Valid idiomatic code should not produce warnings.

😟 problem

• This rule is causing many noisy lint warnings for otherwise valid code and makes it hard to spot real issues.
• Many developers prefer single-line blocks for brevity. Forcing braces creates a large volume of stylistic warnings that drown out real problems.

💡 proposed solutions

• option to accept brace-less single-statement blocks
• or use a less strict default
• or accept both styles by default and optionally enforce braces only for multi-line blocks.
• Or support common linter config formats (ESLint/.editorconfig) to override this rule.

👍 benefits

• Allowing configuration preserves safety-conscious defaults while reducing noise for users who prefer concise style.

[erosman]

Which version of FireMonkey are you using?

• In v2.*, you can customise CodeMirror rules in Options -> Advanced -> CodeMirror
• in v3.0, you can customise ESLint rules in Options -> Advanced -> Linter Options

FireMonkey 2

{
  "jshint": {
    "curly": false
  }
}

FireMonkey 3

In v3.0, ESLint rule for braces is curly i.e

  "curly": "error",

Example of overriding the rule in Linter Options with "multi"

{
  "curly": ["error", "multi"]
}

Example of turning the rule off

{
  "curly": "off"
}

[chrizilla]

Thanks for the help 🙏 and sorry for my apparent blindness.

The CodeMirror syntax works for turning off the curly braces check completely: {"jshint": {"curly": false}}

In Firemonkey 3, I can apparently turn on the curly braces check for multiple statements only: {"curly": ["error", "multi"]}.
Can I achieve this in Firemonkey 2.74 as well somehow ?

[erosman]

Can I achieve this in Firemonkey 2.74 as well somehow ?

@chrizilla Sadly, JSHint doesn’t have that option for curly.