#8723 closed patch
Sort the list of engines
| Reported by: | jvprat | Owned by: | sev- |
|---|---|---|---|
| Priority: | normal | Component: | GUI |
| Version: | Keywords: | ||
| Cc: | Game: |
Description
This patch sorts all the lists of engines alphabetically, like it was done some time ago with the configure help options list.
The main advantadge of this would be for people working on new engines outside of the ScummVM tree, because it would prevent the current conflicts to the "add engine" patches when a new engine is added upstream, because the context would be much less prone to change.
Ticket imported from: #1800767. Ticket imported from: patches/828.
Attachments (2)
Change History (11)
by , 17 years ago
| Attachment: | sort_engines.patch added |
|---|
comment:1 by , 17 years ago
Looks good. I would always set SCUMM to the top though, since it's the 'main' and oldest engine of ScummVM. SCUMM is set to the top of our credits too.
comment:2 by , 17 years ago
That would be reasonable if it's always done the same way. For example: always begin with scumm, scumm7-8, he, and follow with the rest alphabetically. When you decide the rules I can rework the patch.
by , 17 years ago
| Attachment: | sort_engines2.patch added |
|---|
Sort engines leaving SCUMM and variants at the top
comment:3 by , 17 years ago
I'm attaching a second version based on my previous comment. File Added: sort_engines2.patch
comment:4 by , 17 years ago
I agree with lordhoto here. Rest is fine (but I didn't have a chance to look at the patch yet, I am just talking about the general idea here :). Eugene?
comment:5 by , 17 years ago
| Owner: | set to |
|---|
comment:6 by , 17 years ago
Looked at the patch, it seems fine to me. So if Eugene agrees, we can commit this.
comment:8 by , 17 years ago
| Status: | new → closed |
|---|
comment:9 by , 5 years ago
| Component: | → GUI |
|---|
Patch to sort the engines alphabetically in the source