This fixes an issue with the C# linter where a message could be returned from cslint that wasn't intended for use with parameters. This just ensures there's enough parameters so that it won't crash (and consequently ignore lint messages).
Details
Details
- Reviewers
epriestley - Group Reviewers
Blessed Reviewers - Commits
- rARCfcdacc28b8a2: Fix issue where message contained % but wasn't intended for use with parameters
Ran the linter, it didn't crash.
Diff Detail
Diff Detail
- Repository
- rARC Arcanist
- Lint
Lint Skipped - Unit
Tests Skipped
Event Timeline
Comment Actions
Is the "crash" a vsprintf() error raising into an exception? Can we do something like this instead?
$message = @vsprintf($pattern, ...);
if ($message === false) {
$message = $pattern;
}That will get a bunch of (probably hypothetical) cases right that this misses, I think.