Request for review: 8017611: Auto corrector for mistyped vm options

Tao Mao tao.mao at oracle.com
Wed Jun 26 14:19:31 PDT 2013


new webrev:
http://cr.openjdk.java.net/~tamao/8017611/webrev.01/

diff from the last webrev:
(1) put more restrictive type check when passing strings.
(2) move VMOptionsFuzzyMatchSimilarity to be a local constant.
(3) simplify routine str_similar()
(4) add jtreg test

Suggestions are taken from Dmitry, Bengt, Vladimir and Christian. Thank 
you all.

Tao


On 6/25/13 10:44 AM, Tao Mao wrote:
> I'm kinda tired of mistyping VM options and figuring it out 
> "mentally"...let machines do it then.
>
> changeset:
> Currently, when we mistype VM options you will get the warning as 
> following
>
> $java -XX:+UseParalelGC -version
> Unrecognized VM option 'UseParalelGC'
>
> I think we can do one step further to interpret the user's intent and 
> suggest the correct VM options. Then i start to imagine that we have 
> something like this
>
> $java -XX:+UseParalelGC -version
> Unrecognized VM option 'UseParalelGC'
> Did you mean 'UseParallelGC'?
>
> Please refer to the wikipedia 
> http://en.wikipedia.org/wiki/S%C3%B8rensen%E2%80%93Dice_coefficient to 
> get an idea of obtaining string similarity. This new feature is based 
> on this theory.
>
> webrev:
> http://cr.openjdk.java.net/~tamao/8017611/webrev.00/
>
> Please try out this patch and see if it meets some need.
>
> Thanks.
> Tao


More information about the hotspot-dev mailing list