Page 1 of 1

Command-Line Options

Posted: January 12th, 2017, 6:28 pm
by Norbert
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.

Re: Command-Line Options

Posted: January 12th, 2017, 6:42 pm
by oitofelix
Norbert wrote:These two are common on GNU/Linux (info).
If it's of any use, MININIM implements that using GNU C library's argp.
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.
Could you explain why you find that useful? Perhaps it's applicable to MININIM as well.

Re: Command-Line Options

Posted: January 12th, 2017, 7:05 pm
by Norbert
oitofelix wrote:
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.
Could you explain why you find that useful? Perhaps it's applicable to MININIM as well.
I actually thought MININIM already had the option.
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

Posted: January 12th, 2017, 7:32 pm
by oitofelix
Norbert wrote:might make creating replays easier
How so? Do you have any particular situation in mind? An example you may provide?
Norbert wrote:might one day be useful for replicating/testing something.
Indeed. That's reason enough to implement that. Thanks for the suggestion.

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

Posted: January 12th, 2017, 9:35 pm
by oitofelix
Norbert wrote:Maybe add
--version (and -v)
It could simply do:
printf ("SDLPoP %s\n", SDLPOP_VERSION);
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"?

Re: Command-Line Options

Posted: January 13th, 2017, 4:17 am
by oitofelix
Norbert wrote:I actually thought MININIM already had the option.
If it does not, the same suggestion for MININIM.
That has been implemented in the latest MININIM release.

Re: Command-Line Options

Posted: January 15th, 2017, 12:19 pm
by David
Done both the command line and the Ctrl+V: https://github.com/NagyD/SDLPoP/commit/ ... 7b8ecf51d8

Re: Command-Line Options

Posted: January 15th, 2017, 4:53 pm
by Norbert
oitofelix wrote:That has been implemented in the latest MININIM release.
David wrote:Done both the command line and the Ctrl+V: https://github.com/NagyD/SDLPoP/commit/ ... 7b8ecf51d8
Neat (2x).

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

Posted: January 21st, 2017, 7:50 pm
by David
Norbert wrote:So, is anything in particular stopping version 1.17 from being released?
Just good old laziness. :)

Here it is: viewtopic.php?p=20558#p20558

Re: Command-Line Options

Posted: January 22nd, 2017, 1:09 pm
by Falcury
Norbert wrote:Maybe you want to include Falcury's icon, or?
Should I make a pull request for that?

Edit:
I opened a pull request. But, feel free to close it again if you think it's better not to use that icon.