Changes between Initial Version and Version 1 of Ticket #12975, comment 1


Ignore:
Timestamp:
Oct 2, 2021, 11:41:19 AM (3 years ago)
Author:
criezy

Legend:

Unmodified
Added
Removed
Modified

  • Ticket #12975, comment 1

    initial v1  
    11I 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.
    22
    3 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.
     3And 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 `DefaultSaveFileManager` so that you can reimplement it for the WinCE and Symbian backends.