/hg/icedtea-web: IcedTea-Web Splashscreen allows zero or one hyp...

Jiri Vanek jvanek at redhat.com
Tue Oct 7 11:27:24 UTC 2014


> I'll review it right now, It looks good :)

Thanx!
> The only thing I'm wondering is maybe instead of sanitize call the method removeAllDashes / removeDashes / sanitizeDashes to be more descriptive of what it does.

renamed
>
> Other than that it looks good to push if you want to go into the direction of supporting -*-headless :) +1


Yes, I really would like

My reason is:
  - the linux convention is -x where x is short argument. and -abcd are four short args - we dont 
support this
  - another convention is --longOne  two hyphens means long, verbose argument - now we are 
supporting it :)
  - for some reason, pilot javaws impl was not using neither -  it had -longOne :-/

So imho now we are more unix like, and still like pilot javaws....


my $0.02
>
>> Sorry for the "attack", but at least it clarified the  -/--/---// hyphens
>> issue
>
Thanx for check!
  pushed.

>



More information about the distro-pkg-dev mailing list