[KAutoSaveFile] Improve the comment in tempFileName()
ClosedPublic

Authored by ahmadsamir on Dec 14 2019, 8:57 AM.

Details

Summary

Hopefully now it's clear that the filename length in this context is the
filename component only, e.g. test.txt, and not the length of the whole
path, e.g. /home/timon/test.txt. (I think part of the confusion probably
comes from the fact that from the POV of a filesystem a dir is a file with
special attributes...).

Diff Detail

Repository
R244 KCoreAddons
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
ahmadsamir created this revision.Dec 14 2019, 8:57 AM
Restricted Application added a project: Frameworks. · View Herald TranscriptDec 14 2019, 8:57 AM
ahmadsamir requested review of this revision.Dec 14 2019, 8:57 AM
dfaure accepted this revision.Dec 14 2019, 9:40 AM

Much better indeed. filesize sounds like QFileInfo::size().

This revision is now accepted and ready to land.Dec 14 2019, 9:40 AM
This revision was automatically updated to reflect the committed changes.