public inbox for frysk-cvs@sourceware.org
help / color / mirror / Atom feed
From: pmuldoon@sourceware.org
To: frysk-cvs@sourceware.org
Subject: [SCM]  master: Print option help if Commands are 1 or more, not more than 1.
Date: Thu, 10 Jan 2008 16:54:00 -0000	[thread overview]
Message-ID: <20080110165446.14138.qmail@sourceware.org> (raw)

The branch, master has been updated
       via  dae63f3724c8a8eb9a5d92d0aa9df4bbd809ac3e (commit)
       via  75ddb8c7d72962dfd1a818a652dda27ffc62029d (commit)
      from  87bc11379df13e358e67a9c37206c3d52f13a5eb (commit)

Those revisions listed above that are new to this repository have
not appeared on any other notification email.

- Log -----------------------------------------------------------------
commit dae63f3724c8a8eb9a5d92d0aa9df4bbd809ac3e
Merge: 75ddb8c7d72962dfd1a818a652dda27ffc62029d 87bc11379df13e358e67a9c37206c3d52f13a5eb
Author: Phil Muldoon <pmuldoon@redhat.com>
Date:   Thu Jan 10 16:54:07 2008 +0000

    Print option help if Commands are 1 or more, not more than 1.
    
    2008-01-10  Phil Muldoon  <pmuldoon@redhat.com>
    
    	* ParameterizedCommand.java (help): Print help if longCommands >0 ! >1.
    	Seperate options from help text.
    
    Merge branch 'master' of ssh://sources.redhat.com/git/frysk

commit 75ddb8c7d72962dfd1a818a652dda27ffc62029d
Author: Phil Muldoon <pmuldoon@redhat.com>
Date:   Thu Jan 10 16:53:15 2008 +0000

    Print out command help if there is only one option (previously > 1).
    
    2008-01-10  Phil Muldoon  <pmuldoon@redhat.com>
    
    	* ParameterizedCommand.java (help): Print help if longCommands >0 ! >1.
    	Seperate options from help text.

-----------------------------------------------------------------------

Summary of changes:
 frysk-core/frysk/hpd/ChangeLog                 |    3 +++
 frysk-core/frysk/hpd/ParameterizedCommand.java |    4 ++--
 2 files changed, 5 insertions(+), 2 deletions(-)

First 500 lines of diff:
diff --git a/frysk-core/frysk/hpd/ChangeLog b/frysk-core/frysk/hpd/ChangeLog
index 6cc7e26..8adafc8 100644
--- a/frysk-core/frysk/hpd/ChangeLog
+++ b/frysk-core/frysk/hpd/ChangeLog
@@ -1,5 +1,8 @@
 2008-01-10  Phil Muldoon  <pmuldoon@redhat.com>
 
+	* ParameterizedCommand.java (help): Print help if longCommands >0 ! >1.
+	Seperate options from help text.
+	 
 	* TestRegs.java (testRegsCommand): Add general-regs
 	* RegsCommand.java: (RegsCommand): New constructor.
 	(interpret): Account for new constructor setting register
diff --git a/frysk-core/frysk/hpd/ParameterizedCommand.java b/frysk-core/frysk/hpd/ParameterizedCommand.java
index eba9dfb..0360746 100644
--- a/frysk-core/frysk/hpd/ParameterizedCommand.java
+++ b/frysk-core/frysk/hpd/ParameterizedCommand.java
@@ -144,8 +144,8 @@ abstract class ParameterizedCommand extends Command {
 
     void help(CLI cli, Input input) {
 	cli.outWriter.print(syntax);
-	if (longOptions.size() > 1) {
-	    cli.outWriter.print(" -option ...; where options are:");
+	if (longOptions.size() > 0) {
+	    cli.outWriter.println(" -option ...; where options are:");
 	    for (Iterator i = longOptions.values().iterator();
 		 i.hasNext(); ) {
 		CommandOption option = (CommandOption)i.next();


hooks/post-receive
--
frysk system monitor/debugger


                 reply	other threads:[~2008-01-10 16:54 UTC|newest]

Thread overview: [no followups] expand[flat|nested]  mbox.gz  Atom feed

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20080110165446.14138.qmail@sourceware.org \
    --to=pmuldoon@sourceware.org \
    --cc=frysk-cvs@sourceware.org \
    --cc=frysk@sourceware.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).