more msftidy checks

More msftidy checks.

See
#4369
#4362
#3853

There are some fp like
`def prev_error_log_path; datastore['ERROR_LOG']; end `

but i think those should be changed to multiline for codestyle (#4372)
I'd restrict this to only comments that look like the true offenders, like:

```
  test = @source.scan(/^\#.*http\/\//)
```