[Scummvm-tracker] [ScummVM :: Bugs] #12975: BASE: FSNode should provide also a "remove" method.
ScummVM :: Bugs
trac at scummvm.org
Sat Oct 2 11:40:49 UTC 2021
#12975: BASE: FSNode should provide also a "remove" method.
--------------------------+---------------------
Reporter: carlo-bramini | Owner: (none)
Type: defect | Status: new
Priority: normal | Component: Common
Version: | Resolution:
Keywords: | Game:
--------------------------+---------------------
Comment (by criezy):
I am not sure adding a `remove` method fo `FSNode` is a good idea. This
would open the door to the possibility to remove any file on the file
system if misused in engines for example, and could thus be dangerous. I
think keeping the file removal restricted to the save file folder might be
a good idea.
And since backends can easily instantiate their own custom SaveFileManager
in `initBackend()` (as is done in the Windows backend for example), a
better approach could be to add a protected virtual `removeFile` method in
`SaveFileManager` so that you can reimplement it for the WinCE and Symbian
backends.
--
Ticket URL: <https://bugs.scummvm.org/ticket/12975#comment:1>
ScummVM :: Bugs <https://bugs.scummvm.org>
ScummVM
More information about the Scummvm-tracker
mailing list