Maybe add
--version (and -v)
It could simply do:
printf ("SDLPoP %s\n", SDLPOP_VERSION);
Maybe add
--help (and -h and -?)
It could simply do:
printf ("See doc/Readme.txt\n");
These two are common on GNU/Linux (info).
Maybe also add
seed=NR
It could force the program to use the supplied random seed.
The latter might make creating replays easier.
Command-Line Options
Re: Command-Line Options
If it's of any use, MININIM implements that using GNU C library's argp.Norbert wrote:These two are common on GNU/Linux (info).
Could you explain why you find that useful? Perhaps it's applicable to MININIM as well.Norbert wrote:Maybe also add
seed=NR
It could force the program to use the supplied random seed.
The latter might make creating replays easier.
Re: Command-Line Options
I actually thought MININIM already had the option.oitofelix wrote:Could you explain why you find that useful? Perhaps it's applicable to MININIM as well.Norbert wrote:Maybe also add
seed=NR
It could force the program to use the supplied random seed.
The latter might make creating replays easier.
If it does not, the same suggestion for MININIM.
Why: might make creating replays easier; might one day be useful for replicating/testing something.
Re: Command-Line Options
How so? Do you have any particular situation in mind? An example you may provide?Norbert wrote:might make creating replays easier
Indeed. That's reason enough to implement that. Thanks for the suggestion.Norbert wrote:might one day be useful for replicating/testing something.
PS: Of course, this is only useful for implementations that make careful use of their random number generators. Otherwise, that won't guarantee reproducibility. That should be the case for any implementation having replays, though.
Re: Command-Line Options
By the way, in SDLPoP, CTRL+V prints "PRINCE OF PERSIA V1.0". That's not accurate. Wouldn't it be better to show "SDLPOP V1.17"?Norbert wrote:Maybe add
--version (and -v)
It could simply do:
printf ("SDLPoP %s\n", SDLPOP_VERSION);
Re: Command-Line Options
That has been implemented in the latest MININIM release.Norbert wrote:I actually thought MININIM already had the option.
If it does not, the same suggestion for MININIM.
Re: Command-Line Options
Done both the command line and the Ctrl+V: https://github.com/NagyD/SDLPoP/commit/ ... 7b8ecf51d8
Re: Command-Line Options
oitofelix wrote:That has been implemented in the latest MININIM release.
Neat (2x).David wrote:Done both the command line and the Ctrl+V: https://github.com/NagyD/SDLPoP/commit/ ... 7b8ecf51d8
So, is anything in particular stopping version 1.17 from being released?
Maybe you want to include Falcury's icon, or?
Re: Command-Line Options
Just good old laziness.Norbert wrote:So, is anything in particular stopping version 1.17 from being released?
Here it is: viewtopic.php?p=20558#p20558
Re: Command-Line Options
Should I make a pull request for that?Norbert wrote:Maybe you want to include Falcury's icon, or?
Edit:
I opened a pull request. But, feel free to close it again if you think it's better not to use that icon.