[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