#7982 closed patch
command line option to load a save slot
Reported by: | SF/khalek_ | Owned by: | SF/ender |
---|---|---|---|
Priority: | normal | Component: | --Other-- |
Version: | Keywords: | ||
Cc: | Game: |
Description
this patch changes the list of arguments printed by scummvm to be in alphabetical order and enables the user to choose a game slot to load (defaulting to 0). Also removes _restore as it was unused.
Ticket imported from: #583688. Ticket imported from: patches/87.
Attachments (1)
Change History (7)
by , 22 years ago
Attachment: | loadpatch.txt added |
---|
comment:1 by , 22 years ago
Alphabetical Order in the list of arguments I think is a "Bad Thing" to do. It would have a meaning if someone ALREADY knew the option and was just trying to find the exact syntax in the list (in which case grep would be better), but this is hardly the case. The more convinient way for options ordering I think it would be a logical grouping of options in addition to a sorting based on the importance of an option. For example, -v should be the last printed, and -t, -m, -s, -e should be listed one after the other.
comment:2 by , 22 years ago
Owner: | set to |
---|
comment:3 by , 22 years ago
I am not at home, so I can't verify this, but I believe Painelf already applied this patch to CVS. Endy, can you please check and close this item?
Any reasons why not more people (all?) can be made assignable for the trackers, i.e. techincans - this way, if we get say a patch/bug for PocketPC/MorphoOS etc. we can just assign it directly to the responsible porter.
Also, it might be helpful to add a scummvm-trackers mailing list that gets a mail whenever a bugreport/patch etc is added, I did that for Fink.
comment:4 by , 22 years ago
Status: | new → closed |
---|
comment:5 by , 22 years ago
Has been applied. Closing.
Fingolfin:
I will set up the mailing list presently, that's a good idea. As for adding
more people, I think I will eventuallyl.. I just don't have the patience to go
through and set everyones tracker permissions up :)
comment:6 by , 6 years ago
Component: | → --Other-- |
---|
patch