[Scummvm-tracker] [ScummVM :: Bugs] #12977: Engines: Incorrect MetaEngine::getSavegameFile implementations

[ScummVM :: Bugs]

#12977: Engines: Incorrect MetaEngine::getSavegameFile implementations
-------------------+-----------------------
Reporter:  criezy  |      Owner:  (none)
    Type:  defect  |     Status:  new
Priority:  normal  |  Component:  --Unset--
 Version:          |   Keywords:
    Game:          |
-------------------+-----------------------
 The default MetaEngine::getSavegameFile one of two schemes:
  * `target.####` if `kSavesUseExtendedFormat` is supported
  * `target.s##` otherwise

 Yet many engines do not support `kSavesUseExtendedFormat` and do not
 reimplement `getSavegameFile`, but are still using the first scheme.

 For example with the lure engine, `getSavegameFile(1, "lure")` returns
 `lure.s01`, but the name actually used by the engine is `lure.001`.

 As a result functions using `getSavegameFile()`, such as
 `copySaveFileToFreeSlot()` fail.

 Having a correct implementation for all engines would also allow to
 simplify some code, such as `removeSaveState()` which would no longer need
 to be reimplemented in many engines.
-- 
Ticket URL: <https://bugs.scummvm.org/ticket/12977>
ScummVM :: Bugs <https://bugs.scummvm.org>
ScummVM