Fixed a few things that didn't work well

mineserve

@@ -13,25 +13,23 @@ status() {
     echo
 }
 
-serve() {
-    java -Xms1G -Xmx5G -jar server.jar nogui
-}
-
 start() {
     if test -f "$PID"; then
         echo "Server already running with pid: $(cat $PID)"
         status
         exit 0
     fi
-    screen -S minecraft -dm serve
+    screen -S minecraft -dm java -Xms1G -Xmx5G -jar server.jar nogui
     pgrep -P "$(screen -list | sed -E -n 's/\s*([0-9]+).minecraft.*/\1/p')" > $PID
 }
 
 stop() {
     if test -f "$PID"; then
+        SCREEN_PID=$(screen -list | sed -E -n 's/\s*([0-9]+).minecraft.*/\1/p')
         screen -S minecraft -X stuff "stop␍"
         echo -n "Waiting for shutdown..."
-        while screen -list | grep -q "minecraft"; do
+        # kill -0 checks if a process is running
+        while kill -0 $SCREEN_PID 2> /dev/null; do
             sleep 3
             echo -n "."
         done