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