#264 weed out extraneous help keys

closed-accepted
nobody
None
5
2006-07-07
2006-06-27
Dan Sebald
No

I've gone through all the .trm files and removed
extraneous help keys now that gnuplot can complete all
words in a help string, not just the last word.

...

Just wondering... So many of these help search strings
have this form:

"?commands set terminal png",
"?set terminal png",
"?terminal png",
"?png",

I wonder if that couldn't simply be reduced to just the
first string with a smart enough algorithm. Probably,
but I wonder if it would create possible ambiguities.
There is a bit of reluctance to jump into this sort of
thing because taking out lines is easy. Putting them
back in isn't so easy.

One way to deal with ambiguities might be to include in
the string the question mark before every word at which
level should be included. For example, say the above
could be condensed as:

"?commands ?set ?terminal ?png",

and the key building code know to construct the four
keys above.

If one level was to be left out for some reason:

"?commands set ?terminal ?png",

would find:

commands set terminal png
terminal png
png

but not

set terminal png

Dan

Discussion

  • Dan Sebald

    Dan Sebald - 2006-06-27

    Logged In: YES
    user_id=704782

    Here are some suggested edits for gnuplot.doc as well. Use
    gvim to hilight the suggested changes in the diff file and
    remove any hunks you don't agree with. Then patch.

     
  • Ethan Merritt

    Ethan Merritt - 2006-07-06
    • priority: 5 --> 2
     
  • Petr Mikulik

    Petr Mikulik - 2006-07-06

    Logged In: YES
    user_id=31505

    gnuplotdoc-djs-27jun2006.patch.gz ... fine with me; I
    propose to add it to cvs, while also changing help for
    defined(): "deprecated" => "deprecated, replaced by `exists`".

    Hm, I've noticed that Octave/Matlab is using "if
    exist('a')", while gnuplot "if exists('a')" ... yes, that
    "exists()" was may suggestion based on Octave with that
    small mistake ... shouldn't we change it in gnuplot into
    "exist()"?

    The other patch, weedterm-djs-27jun2006.patch.gz, looks also OK.

     
  • Petr Mikulik

    Petr Mikulik - 2006-07-06
    • priority: 2 --> 5
     
  • Ethan Merritt

    Ethan Merritt - 2006-07-07
    • status: open --> closed-accepted
     

Log in to post a comment.

Get latest updates about Open Source Projects, Conferences and News.

Sign up for the SourceForge newsletter:





No, thanks