README.md

@@ -49,7 +49,7 @@ need it
 - `screen` - for process supervision
 - `source-highlight` - for syntax highlighting when displaying launching
 scripts
-- any text editor - used for builtin function to edit launching scripts
+- any text editor - used for builting function to edit launching scripts
 
 ## Installation instructions
 

bin/qemush

@@ -93,7 +93,7 @@ public_diskls() {
 public_edit() {
     local file="${bin}/${1}"
     "$EDITOR" "$file"
-    [ -f "$file" ] && exec chmod u+x "$file"
+    exec chmod u+x "$file"
 }
 
 public_rm() {

src/first-free-port.c

@@ -3,7 +3,6 @@
 #include <fcntl.h>
 #include <sysexits.h>
 #include <string.h>
-#include <errno.h>
 
 #define MIN_TCP_PORT 1
 #define TCP_TABLE "/proc/net/tcp"
@@ -13,13 +12,6 @@
 #define LISTENING_HEX "0A"
 #define PORTS_BLOCKS_TO_ALLOW 4
 
-/**
- * Print a nice error message
- */
-static void print_error(char *name, const char *message) {
-    fprintf(stderr, "%s: %s\n", name, message);
-}
-
 /**
  * Get all the listening TCP ports
  */
@@ -89,25 +81,20 @@ int main(int argc, char *argv[]) {
     }
 
     if (!is_valid_tcp_port(current_port)) {
-        print_error(argv[0], "provide a valid TCP port number as first argument.");
+        fprintf(stderr, "Provide a valid TCP port number as first argument.\n");
         return EX_USAGE;
     }
 
     // Open TCP table
     FILE *tcp_table_fptr = fopen(TCP_TABLE, "r");
     if (tcp_table_fptr == NULL) {
-        print_error(argv[0], "error opening the TCP table.");
-        return errno;
+        fprintf(stderr, "Error opening the TCP table.\n");
+        return EX_OSFILE;
     }
 
     unsigned short *listening_ports = malloc(PORTS_BLOCKS_TO_ALLOW * sizeof(unsigned short));
     listening_ports = get_listening_ports(listening_ports, tcp_table_fptr);
 
-    if (fclose(tcp_table_fptr) != 0) {
-        print_error(argv[0], "can't close the TCP table.");
-        return errno;
-    }
-
     // Check if the current port is available, add
     while (!is_port_available(current_port, listening_ports)) {
         current_port++;
@@ -119,7 +106,7 @@ int main(int argc, char *argv[]) {
         printf("%d\n", current_port);
         return EXIT_SUCCESS;
     } else {
-        print_error(argv[0], "no more ports available; how did you fuck up that bad ???");
+        fprintf(stderr, "No more ports available. How did you fuck up that bad ???\n");
         return EX_TEMPFAIL;
     }
 }