chore: address review comments

src/main.js

@@ -75,7 +75,7 @@ async function main(filename, resourceName, resource, options) {
             `Segment and section with that name already exists: ${options.machoSegmentName}/${sectionName}`
           );
           console.log("Use --overwrite to overwrite the existing content");
-          process.exit(2);
+          process.exit(1);
         }
       }
       break;
@@ -96,7 +96,7 @@ async function main(filename, resourceName, resource, options) {
         if (result === postject.InjectResult.kAlreadyExists) {
           console.log(`Section with that name already exists: ${sectionName}`);
           console.log("Use --overwrite to overwrite the existing content");
-          process.exit(2);
+          process.exit(1);
         }
       }
       break;
@@ -118,15 +118,15 @@ async function main(filename, resourceName, resource, options) {
             `Resource with that name already exists: ${resourceName}`
           );
           console.log("Use --overwrite to overwrite the existing content");
-          process.exit(2);
+          process.exit(1);
         }
       }
       break;
   }
 
   if (result !== postject.InjectResult.kSuccess) {
     console.log("Error when injecting resource");
-    process.exit(3);
+    process.exit(1);
   }
 
   try {

src/postject.cpp

@@ -14,9 +14,9 @@ enum class ExecutableFormat { kELF, kMachO, kPE, kUnknown };
 enum class InjectResult { kAlreadyExists, kError, kSuccess };
 
 std::vector<uint8_t> vec_from_val(const emscripten::val& value) {
-  // NOTE - vecFromJSArray incurs a copy, so memory usage is higher than it
-  //        needs to be. Explore ways to access the memory directly and avoid
-  //        the copy.
+  // TODO(dsanders11) - vecFromJSArray incurs a copy, so memory usage is higher
+  //                    than it needs to be. Explore ways to access the memory
+  //                    directly and avoid the copy.
   return emscripten::vecFromJSArray<uint8_t>(value);
 }