Ticket #5167: typo fixes in src/filemanager/file.c

[tuffnatty]

Proposed changes

A typo fix in comment, a typo fix in printf format string

Checklist

• I have referenced the issue(s) resolved by this PR (if any)
• I have signed-off my contribution with git commit --amend -s
• Lint and unit tests pass locally with my changes (make indent && make check)
• I have added tests that prove my fix is effective or that my feature works
• I have added the necessary documentation (if appropriate)

[zyv]

Inside file_error, an empty string is passed as a second format argument, but my understanding is that it is harmless to remove the format descriptor, even though it's a somewhat questionable design.

Online C Draft Standard (n1256), s. 7.19.6.1, p. 2:

The fprintf function writes output to the stream pointed to by stream, under control of the string pointed to by format that specifies how subsequent arguments are converted for output. If there are insufficient arguments for the format, the behavior is undefined. If the format is exhausted while arguments remain, the excess arguments are evaluated (as always) but are otherwise ignored. The fprintf function returns when the end of the format string is encountered.

[tuffnatty]

Sure it's not a bug but seems to be unintentional leftover and looks misleading.

[zyv]

/rebase