2008-02-27 Joel Sherrill <joel.sherrill@oarcorp.com>

	* libmisc/shell/cat_file.c, libmisc/shell/main_alias.c,
	libmisc/shell/main_blksync.c, libmisc/shell/main_cat.c,
	libmisc/shell/main_chdir.c, libmisc/shell/main_chmod.c,
	libmisc/shell/main_chroot.c, libmisc/shell/main_cpuuse.c,
	libmisc/shell/main_date.c, libmisc/shell/main_help.c,
	libmisc/shell/main_id.c, libmisc/shell/main_logoff.c,
	libmisc/shell/main_ls.c, libmisc/shell/main_mallocinfo.c,
	libmisc/shell/main_mdump.c, libmisc/shell/main_medit.c,
	libmisc/shell/main_mfill.c, libmisc/shell/main_mkdir.c,
	libmisc/shell/main_mmove.c, libmisc/shell/main_mount.c,
	libmisc/shell/main_mount_nfs.c, libmisc/shell/main_msdosfmt.c,
	libmisc/shell/main_mwdump.c, libmisc/shell/main_perioduse.c,
	libmisc/shell/main_pwd.c, libmisc/shell/main_rm.c,
	libmisc/shell/main_rmdir.c, libmisc/shell/main_stackuse.c,
	libmisc/shell/main_tty.c, libmisc/shell/main_umask.c,
	libmisc/shell/main_unmount.c, libmisc/shell/main_whoami.c,
	libmisc/shell/shell.h: Clean up done while writing documentation.
	Some command improvements such as date now allows setting of the
	current TOD. Often commands did not use stdout/stderr per
	expectations and did not return -1 on an error.

1 files changed, 24 insertions, 6 deletions

diff --git a/cpukit/libmisc/shell/main_perioduse.c b/cpukit/libmisc/shell/main_perioduse.c
index 349e168cd4..9ad18d76de 100644
--- a/cpukit/libmisc/shell/main_perioduse.c
+++ b/cpukit/libmisc/shell/main_perioduse.c
@@ -21,18 +21,36 @@
 #include <rtems/shell.h>
 #include "internal.h"
 
-int rtems_shell_main_perioduse(int argc,char *argv[])
+int rtems_shell_main_perioduse(
+  int   argc,
+  char *argv[]
+)
 {
-  if ( argc >= 1 && !strcmp( argv[1], "-r" ) ) {
-    printf( "Resetting Period Usage information\n" );
-    rtems_rate_monotonic_reset_all_statistics();
-  } else {
+  /*
+   *  When invoked with no arguments, print the report.
+   */
+  if ( argc == 1 ) {
     rtems_rate_monotonic_report_statistics_with_plugin(
       stdout,
       (rtems_printk_plugin_t)fprintf
     );
+    return 0;
   }
-  return 0;
+
+  /*
+   *  When invoked with the single argument -r, reset the statistics.
+   */
+  if ( argc == 2 && !strcmp( argv[1], "-r" ) ) {
+    printf( "Resetting Period Usage information\n" );
+    rtems_rate_monotonic_reset_all_statistics();
+    return 0;
+  }
+
+  /*
+   *  OK.  The user did something wrong.
+   */
+  fprintf( stderr, "%s: [-r]\n", argv[0] );
+  return -1;
 }
 
 rtems_shell_cmd_t rtems_shell_PERIODUSE_Command = {