Changeset 15301

Timestamp:

Oct 12, 2007, 9:33:15 AM (19 years ago)

Author:

Paul Price

Message:

Some values are not required if we are passing an error code.

Location:

trunk/ippTools/src

Files:

• stacktool.c (modified) (2 diffs)
• stacktoolConfig.c (modified) (1 diff)

trunk/ippTools/src/stacktool.c

@@ -506 +506 @@
         return false;
     }
-    if (!uri) {
-        psError(PS_ERR_UNKNOWN, true, "-uri is required");
-        return false;
-    }
+
     psString path_base = psMetadataLookupStr(&status, config->args, "-path_base");
     if (!status) {
         psError(PS_ERR_UNKNOWN, false, "failed to lookup value for -path_base");
-        return false;
-    }
-    if (!path_base) {
-        psError(PS_ERR_UNKNOWN, true, "-path_base is required");
         return false;
     }
@@ -671 +664 @@
 {
     PS_ASSERT_PTR_NON_NULL(config, false);
-    
+
     psString query = pxDataGet("stacktool_revertsumskyfile.sql");
     if (!query) {

trunk/ippTools/src/stacktoolConfig.c

@@ -109 +109 @@
     psMetadata *addsumskyfileArgs = psMetadataAlloc();
     psMetadataAddStr(addsumskyfileArgs, PS_LIST_TAIL, "-stack_id", 0,
-            "define warp ID (required)", NULL);
+            "define stack ID (required)", NULL);
     psMetadataAddStr(addsumskyfileArgs, PS_LIST_TAIL, "-uri", 0,
-            "define URI of file (required)", 0);
+            "define URI of file", 0);
     psMetadataAddStr(addsumskyfileArgs, PS_LIST_TAIL, "-path_base", 0,
-            "define base output location (required)", 0);
+            "define base output location", 0);
     psMetadataAddF64(addsumskyfileArgs, PS_LIST_TAIL, "-bg",  0,
             "define exposue background", NAN);