taviso / 123elf

A native port of Lotus 1-2-3 to Linux.
1.17k stars 59 forks source link

Rewrite command line parser for Linux adaptation #33

Closed jserv closed 2 years ago

jserv commented 2 years ago

The original command line parser of Lotus 1-2-3 for UNIX supported deprecated and useless features, and there are some outdated options as following:

Meanwhile, we might extend the command line parser to support new options such as --no-banner / --show-banner (#28), --ignore-case (file-name mode ignoring the case of the characters), and --enable-autoexec / --disable-autoexec (#27).

taviso commented 2 years ago

I think this is done now. Originally @jserv wrote a patch that recreated argv, but I removed it because I thought It would be cleaner to reuse the same argv array.

It turned out that was the wrong idea, and the original approach was better. That's implemented now, and I think it gives us some flexibility on adding more parameters in future.