increment PC after every Instruction

de/darkress/pic16f84sim/Main.java

@@ -2,29 +2,22 @@ package de.darkress.pic16f84sim;
 
 import de.darkress.pic16f84sim.commands.Command;
 import de.darkress.pic16f84sim.decoder.CommandDecoder;
-import de.darkress.pic16f84sim.microcontroller.Cycles;
 import de.darkress.pic16f84sim.microcontroller.Memory;
 import de.darkress.pic16f84sim.microcontroller.ProgramCounter;
-import de.darkress.pic16f84sim.parser.Parser;
 
-import java.util.concurrent.TimeUnit;
+import java.util.ArrayList;
 
 class Main
 {
-    public static void main(String[] args) throws InterruptedException {
+    public static void main(String[] args) {
 
-        Command[] program = Parser.parser("de/darkress/pic16f84sim/TestPrograms/TPicSim7.LST");
-
-        Memory.initMemory();
-
-        while(ProgramCounter.getPc() < 1024)
+        ArrayList<Command> program = new ArrayList<>();
+        Memory.workingRegister = 0x01;
+        Memory.setRegister(0x14, 0xA7); //240 << 224
+        program.add(CommandDecoder.decode(0x1C94));
+        for(int i = 0; i < program.size(); i++)
         {
-            System.out.println("Command: " + program[ProgramCounter.getPc()].toString());
-            program[ProgramCounter.getPc()].execute();
-
-            System.out.println(Integer.toHexString(Memory.workingRegister) + " " + Cycles.getCycles());
-            System.out.println(Integer.toHexString(Memory.getOption()) + " " + Integer.toHexString(Memory.getTimer()));
-            System.out.println(Integer.toHexString(Memory.getPCLATH()) + " " + Integer.toHexString(Memory.getPCL()) + "\n");
+            program.get(ProgramCounter.getPc()).execute();
         }
     }
 }

de/darkress/pic16f84sim/commands/Addlw.java

@@ -1,6 +1,5 @@
 package de.darkress.pic16f84sim.commands;
 
-import de.darkress.pic16f84sim.microcontroller.Cycles;
 import de.darkress.pic16f84sim.microcontroller.Memory;
 import de.darkress.pic16f84sim.microcontroller.ProgramCounter;
 
@@ -16,8 +15,6 @@ public class Addlw extends LiteralCommandUtils implements Command
     @Override
     public void execute()
     {
-        ProgramCounter.incPC();
-        Cycles.incCycles();
         int result = literal + Memory.workingRegister;
 
         checkZeroBit(result);
@@ -25,5 +22,6 @@ public class Addlw extends LiteralCommandUtils implements Command
         checkDigitCarryBit(literal);
 
         Memory.workingRegister = result % 256;
+        ProgramCounter.incPC();
     }
 }

de/darkress/pic16f84sim/commands/Addwf.java

@@ -1,6 +1,5 @@
 package de.darkress.pic16f84sim.commands;
 
-import de.darkress.pic16f84sim.microcontroller.Cycles;
 import de.darkress.pic16f84sim.microcontroller.Memory;
 import de.darkress.pic16f84sim.microcontroller.ProgramCounter;
 
@@ -18,8 +17,6 @@ public class Addwf extends FileRegisterCommandUtils implements Command
     @Override
     public void execute()
     {
-        ProgramCounter.incPC();
-        Cycles.incCycles();
         int result = Memory.getRegister(address) + Memory.workingRegister;
 
         checkZeroBit(result);
@@ -27,5 +24,6 @@ public class Addwf extends FileRegisterCommandUtils implements Command
         checkDigitCarryBit(address);
 
         writeToDestination(destinationBit, address, result);
+        ProgramCounter.incPC();
     }
 }

de/darkress/pic16f84sim/commands/Andlw.java

@@ -1,6 +1,5 @@
 package de.darkress.pic16f84sim.commands;
 
-import de.darkress.pic16f84sim.microcontroller.Cycles;
 import de.darkress.pic16f84sim.microcontroller.Memory;
 import de.darkress.pic16f84sim.microcontroller.ProgramCounter;
 
@@ -15,12 +14,11 @@ public class Andlw extends LiteralCommandUtils implements Command
     @Override
     public void execute()
     {
-        ProgramCounter.incPC();
-        Cycles.incCycles();
         int result = literal & Memory.workingRegister;
 
         checkZeroBit(result);
 
         Memory.workingRegister = result % 256;
+        ProgramCounter.incPC();
     }
 }

de/darkress/pic16f84sim/commands/Andwf.java

@@ -1,6 +1,5 @@
 package de.darkress.pic16f84sim.commands;
 
-import de.darkress.pic16f84sim.microcontroller.Cycles;
 import de.darkress.pic16f84sim.microcontroller.Memory;
 import de.darkress.pic16f84sim.microcontroller.ProgramCounter;
 
@@ -18,12 +17,11 @@ public class Andwf extends FileRegisterCommandUtils implements Command
     @Override
     public void execute()
     {
-        ProgramCounter.incPC();
-        Cycles.incCycles();
         int result = Memory.getRegister(address) & Memory.workingRegister;
 
         checkZeroBit(result);
 
         writeToDestination(destinationBit, address, result);
+        ProgramCounter.incPC();
     }
 }

de/darkress/pic16f84sim/commands/Bcf.java

@@ -1,6 +1,5 @@
 package de.darkress.pic16f84sim.commands;
 
-import de.darkress.pic16f84sim.microcontroller.Cycles;
 import de.darkress.pic16f84sim.microcontroller.Memory;
 import de.darkress.pic16f84sim.microcontroller.ProgramCounter;
 
@@ -18,11 +17,10 @@ public class Bcf extends BitOrientedCommandUtils implements Command
     @Override
     public void execute()
     {
-        ProgramCounter.incPC();
-        Cycles.incCycles();
         int result = Memory.getRegister(address);
         result &= ~(1 << bitPlacement); //Mask n-th bit with 0
 
         Memory.setRegister(address, result);
+        ProgramCounter.incPC();
     }
 }

de/darkress/pic16f84sim/commands/Bsf.java

@@ -1,6 +1,5 @@
 package de.darkress.pic16f84sim.commands;
 
-import de.darkress.pic16f84sim.microcontroller.Cycles;
 import de.darkress.pic16f84sim.microcontroller.Memory;
 import de.darkress.pic16f84sim.microcontroller.ProgramCounter;
 
@@ -18,11 +17,10 @@ public class Bsf extends BitOrientedCommandUtils implements Command
     @Override
     public void execute()
     {
-        ProgramCounter.incPC();
-        Cycles.incCycles();
         int result = Memory.getRegister(address);
         result |= (1 << bitPlacement);
 
         Memory.setRegister(address, result);
+        ProgramCounter.incPC();
     }
 }

de/darkress/pic16f84sim/commands/Btfsc.java

@@ -1,6 +1,5 @@
 package de.darkress.pic16f84sim.commands;
 
-import de.darkress.pic16f84sim.microcontroller.Cycles;
 import de.darkress.pic16f84sim.microcontroller.Memory;
 import de.darkress.pic16f84sim.microcontroller.ProgramCounter;
 
@@ -18,14 +17,13 @@ public class Btfsc extends BitOrientedCommandUtils implements Command
     @Override
     public void execute()
     {
-        ProgramCounter.incPC();
-        Cycles.incCycles();
         int result = Memory.getRegister(address);
 
         if((result & (1 << bitPlacement)) == 0) //Test if bit is clear
         {
-            Nop nop = new Nop();
-            nop.execute();
+            ProgramCounter.incPC();
         }
+
+        ProgramCounter.incPC();
     }
 }

de/darkress/pic16f84sim/commands/Btfss.java

@@ -1,6 +1,5 @@
 package de.darkress.pic16f84sim.commands;
 
-import de.darkress.pic16f84sim.microcontroller.Cycles;
 import de.darkress.pic16f84sim.microcontroller.Memory;
 import de.darkress.pic16f84sim.microcontroller.ProgramCounter;
 
@@ -18,14 +17,13 @@ public class Btfss extends BitOrientedCommandUtils implements Command
     @Override
     public void execute()
     {
-        ProgramCounter.incPC();
-        Cycles.incCycles();
         int result = Memory.getRegister(address);
 
         if((result & (1 << bitPlacement)) > 0) //Test if bit is set
         {
-            Nop nop = new Nop();
-            nop.execute();
+            ProgramCounter.incPC();
         }
+
+        ProgramCounter.incPC();
     }
 }

de/darkress/pic16f84sim/commands/Call.java

@@ -1,6 +1,5 @@
 package de.darkress.pic16f84sim.commands;
 
-import de.darkress.pic16f84sim.microcontroller.Cycles;
 import de.darkress.pic16f84sim.microcontroller.ProgramCounter;
 import de.darkress.pic16f84sim.microcontroller.Stack;
 
@@ -16,9 +15,7 @@ public class Call extends LiteralCommandUtils implements Command
     @Override
     public void execute()
     {
-        Cycles.incCycles();
         Stack.push(ProgramCounter.getPc() + 1);
         ProgramCounter.setPcFrom11BitLiteral(literal);
-        Cycles.incCycles(); //Simulate nop and 2-Cycle instruction
     }
 }

de/darkress/pic16f84sim/commands/Clrf.java

@@ -1,6 +1,6 @@
 package de.darkress.pic16f84sim.commands;
 
-import de.darkress.pic16f84sim.microcontroller.Cycles;
+import de.darkress.pic16f84sim.microcontroller.Memory;
 import de.darkress.pic16f84sim.microcontroller.ProgramCounter;
 
 public class Clrf extends FileRegisterCommandUtils implements Command
@@ -17,12 +17,11 @@ public class Clrf extends FileRegisterCommandUtils implements Command
     @Override
     public void execute()
     {
-        ProgramCounter.incPC();
-        Cycles.incCycles();
         final int result = 0;
 
         checkZeroBit(result);
 
         writeToDestination(destinationBit, address, result);
+        ProgramCounter.incPC();
     }
 }

de/darkress/pic16f84sim/commands/Clrw.java

@@ -1,6 +1,5 @@
 package de.darkress.pic16f84sim.commands;
 
-import de.darkress.pic16f84sim.microcontroller.Cycles;
 import de.darkress.pic16f84sim.microcontroller.Memory;
 import de.darkress.pic16f84sim.microcontroller.ProgramCounter;
 
@@ -9,12 +8,11 @@ public class Clrw extends FileRegisterCommandUtils implements Command
     @Override
     public void execute()
     {
-        ProgramCounter.incPC();
-        Cycles.incCycles();
         final int result = 0;
 
         checkZeroBit(result);
 
         Memory.workingRegister = result;
+        ProgramCounter.incPC();
     }
 }

de/darkress/pic16f84sim/commands/Comf.java

@@ -1,6 +1,5 @@
 package de.darkress.pic16f84sim.commands;
 
-import de.darkress.pic16f84sim.microcontroller.Cycles;
 import de.darkress.pic16f84sim.microcontroller.Memory;
 import de.darkress.pic16f84sim.microcontroller.ProgramCounter;
 
@@ -18,12 +17,11 @@ public class Comf extends FileRegisterCommandUtils implements Command
     @Override
     public void execute()
     {
-        ProgramCounter.incPC();
-        Cycles.incCycles();
         int result = 255 - Memory.getRegister(address); // Get inverse of 8Bit value
 
         checkZeroBit(result);
 
         writeToDestination(destinationBit, address, result);
+        ProgramCounter.incPC();
     }
 }

de/darkress/pic16f84sim/commands/Decf.java

@@ -1,6 +1,5 @@
 package de.darkress.pic16f84sim.commands;
 
-import de.darkress.pic16f84sim.microcontroller.Cycles;
 import de.darkress.pic16f84sim.microcontroller.Memory;
 import de.darkress.pic16f84sim.microcontroller.ProgramCounter;
 
@@ -18,12 +17,11 @@ public class Decf extends FileRegisterCommandUtils implements Command
     @Override
     public void execute()
     {
-        ProgramCounter.incPC();
-        Cycles.incCycles();
         int result = Memory.getRegister(address) + 255; // Allow underflow
 
         checkZeroBit(result);
 
         writeToDestination(destinationBit, address, result % 256); // Catch underflow
+        ProgramCounter.incPC();
     }
 }

de/darkress/pic16f84sim/commands/Decfsz.java

@@ -1,6 +1,5 @@
 package de.darkress.pic16f84sim.commands;
 
-import de.darkress.pic16f84sim.microcontroller.Cycles;
 import de.darkress.pic16f84sim.microcontroller.Memory;
 import de.darkress.pic16f84sim.microcontroller.ProgramCounter;
 
@@ -18,8 +17,6 @@ public class Decfsz extends FileRegisterCommandUtils implements Command
     @Override
     public void execute()
     {
-        ProgramCounter.incPC();
-        Cycles.incCycles();
         int result = Memory.getRegister(address) + 255; // Allow underflow
 
         checkZeroBit(result);
@@ -28,8 +25,8 @@ public class Decfsz extends FileRegisterCommandUtils implements Command
 
         if((result % 256) == 0)
         {
-            Nop nop = new Nop();
-            nop.execute();
+            ProgramCounter.incPC();
         }
+        ProgramCounter.incPC();
     }
 }

de/darkress/pic16f84sim/commands/Goto.java

@@ -1,6 +1,5 @@
 package de.darkress.pic16f84sim.commands;
 
-import de.darkress.pic16f84sim.microcontroller.Cycles;
 import de.darkress.pic16f84sim.microcontroller.ProgramCounter;
 
 public class Goto extends LiteralCommandUtils implements Command
@@ -15,8 +14,6 @@ public class Goto extends LiteralCommandUtils implements Command
     @Override
     public void execute()
     {
-        Cycles.incCycles();
         ProgramCounter.setPcFrom11BitLiteral(literal);
-        Cycles.incCycles();
     }
 }

de/darkress/pic16f84sim/commands/Incf.java

@@ -1,6 +1,5 @@
 package de.darkress.pic16f84sim.commands;
 
-import de.darkress.pic16f84sim.microcontroller.Cycles;
 import de.darkress.pic16f84sim.microcontroller.Memory;
 import de.darkress.pic16f84sim.microcontroller.ProgramCounter;
 
@@ -18,12 +17,11 @@ public class Incf extends FileRegisterCommandUtils implements Command
     @Override
     public void execute()
     {
-        ProgramCounter.incPC();
-        Cycles.incCycles();
         int result = Memory.getRegister(address) + 1; // Allow underflow
 
         checkZeroBit(result);
 
         writeToDestination(destinationBit, address, result % 256); // Catch underflow
+        ProgramCounter.incPC();
     }
 }

de/darkress/pic16f84sim/commands/Incfsz.java

@@ -1,6 +1,5 @@
 package de.darkress.pic16f84sim.commands;
 
-import de.darkress.pic16f84sim.microcontroller.Cycles;
 import de.darkress.pic16f84sim.microcontroller.Memory;
 import de.darkress.pic16f84sim.microcontroller.ProgramCounter;
 
@@ -18,8 +17,6 @@ public class Incfsz extends FileRegisterCommandUtils implements Command
     @Override
     public void execute()
     {
-        ProgramCounter.incPC();
-        Cycles.incCycles();
         int result = Memory.getRegister(address) + 1;
 
         checkZeroBit(result);
@@ -28,8 +25,8 @@ public class Incfsz extends FileRegisterCommandUtils implements Command
 
         if((result % 256) == 0)
         {
-            Nop nop = new Nop();
-            nop.execute();
+            ProgramCounter.incPC();
         }
+        ProgramCounter.incPC();
     }
 }

de/darkress/pic16f84sim/commands/Iorlw.java

@@ -1,6 +1,5 @@
 package de.darkress.pic16f84sim.commands;
 
-import de.darkress.pic16f84sim.microcontroller.Cycles;
 import de.darkress.pic16f84sim.microcontroller.Memory;
 import de.darkress.pic16f84sim.microcontroller.ProgramCounter;
 
@@ -15,12 +14,11 @@ public class Iorlw extends LiteralCommandUtils implements Command
     @Override
     public void execute()
     {
-        ProgramCounter.incPC();
-        Cycles.incCycles();
         int result = literal | Memory.workingRegister;
 
         checkZeroBit(result);
 
         Memory.workingRegister = result % 256;
+        ProgramCounter.incPC();
     }
 }

de/darkress/pic16f84sim/commands/Iorwf.java

@@ -1,6 +1,5 @@
 package de.darkress.pic16f84sim.commands;
 
-import de.darkress.pic16f84sim.microcontroller.Cycles;
 import de.darkress.pic16f84sim.microcontroller.Memory;
 import de.darkress.pic16f84sim.microcontroller.ProgramCounter;
 
@@ -18,12 +17,11 @@ public class Iorwf extends FileRegisterCommandUtils implements Command
     @Override
     public void execute()
     {
-        ProgramCounter.incPC();
-        Cycles.incCycles();
         int result = Memory.getRegister(address) | Memory.workingRegister;
 
         checkZeroBit(result);
 
         writeToDestination(destinationBit, address, result);
+        ProgramCounter.incPC();
     }
 }

de/darkress/pic16f84sim/commands/Movf.java

@@ -1,6 +1,5 @@
 package de.darkress.pic16f84sim.commands;
 
-import de.darkress.pic16f84sim.microcontroller.Cycles;
 import de.darkress.pic16f84sim.microcontroller.Memory;
 import de.darkress.pic16f84sim.microcontroller.ProgramCounter;
 
@@ -18,12 +17,11 @@ public class Movf extends FileRegisterCommandUtils implements Command
     @Override
     public void execute()
     {
-        ProgramCounter.incPC();
-        Cycles.incCycles();
         int result = Memory.getRegister(address);
 
         checkZeroBit(result);
 
         writeToDestination(destinationBit, address, result);
+        ProgramCounter.incPC();
     }
 }

de/darkress/pic16f84sim/commands/Movlw.java

@@ -1,6 +1,5 @@
 package de.darkress.pic16f84sim.commands;
 
-import de.darkress.pic16f84sim.microcontroller.Cycles;
 import de.darkress.pic16f84sim.microcontroller.Memory;
 import de.darkress.pic16f84sim.microcontroller.ProgramCounter;
 
@@ -16,8 +15,7 @@ public class Movlw extends LiteralCommandUtils implements Command
     @Override
     public void execute()
     {
-        ProgramCounter.incPC();
-        Cycles.incCycles();
         Memory.workingRegister = literal;
+        ProgramCounter.incPC();
     }
 }

de/darkress/pic16f84sim/commands/Movwf.java

@@ -1,6 +1,5 @@
 package de.darkress.pic16f84sim.commands;
 
-import de.darkress.pic16f84sim.microcontroller.Cycles;
 import de.darkress.pic16f84sim.microcontroller.Memory;
 import de.darkress.pic16f84sim.microcontroller.ProgramCounter;
 
@@ -18,10 +17,9 @@ public class Movwf extends FileRegisterCommandUtils implements Command
     @Override
     public void execute()
     {
-        ProgramCounter.incPC();
-        Cycles.incCycles();
         int result = Memory.workingRegister;
 
         writeToDestination(destinationBit, address, result);
+        ProgramCounter.incPC();
     }
 }

de/darkress/pic16f84sim/commands/Nop.java

@@ -1,6 +1,5 @@
 package de.darkress.pic16f84sim.commands;
 
-import de.darkress.pic16f84sim.microcontroller.Cycles;
 import de.darkress.pic16f84sim.microcontroller.ProgramCounter;
 
 public class Nop extends LiteralCommandUtils implements Command
@@ -11,6 +10,5 @@ public class Nop extends LiteralCommandUtils implements Command
     {
         // Do nothing, just increment the PC
         ProgramCounter.incPC();
-        Cycles.incCycles();
     }
 }

de/darkress/pic16f84sim/commands/Retlw.java

@@ -1,6 +1,5 @@
 package de.darkress.pic16f84sim.commands;
 
-import de.darkress.pic16f84sim.microcontroller.Cycles;
 import de.darkress.pic16f84sim.microcontroller.Memory;
 import de.darkress.pic16f84sim.microcontroller.ProgramCounter;
 import de.darkress.pic16f84sim.microcontroller.Stack;
@@ -17,9 +16,7 @@ public class Retlw extends LiteralCommandUtils implements Command
     @Override
     public void execute()
     {
-        Cycles.incCycles();
         Memory.workingRegister = literal;
         ProgramCounter.setPcFromStack(Stack.pop());
-        Cycles.incCycles(); // Simulate 2-Cycle Instruction
     }
 }

de/darkress/pic16f84sim/commands/Return.java

@@ -1,6 +1,5 @@
 package de.darkress.pic16f84sim.commands;
 
-import de.darkress.pic16f84sim.microcontroller.Cycles;
 import de.darkress.pic16f84sim.microcontroller.ProgramCounter;
 import de.darkress.pic16f84sim.microcontroller.Stack;
 
@@ -9,8 +8,6 @@ public class Return extends LiteralCommandUtils implements Command
     @Override
     public void execute()
     {
-        Cycles.incCycles();
         ProgramCounter.setPcFromStack(Stack.pop());
-        Cycles.incCycles(); // Simulate 2-Cycle Instruction
     }
 }

de/darkress/pic16f84sim/commands/Rlf.java

@@ -1,6 +1,5 @@
 package de.darkress.pic16f84sim.commands;
 
-import de.darkress.pic16f84sim.microcontroller.Cycles;
 import de.darkress.pic16f84sim.microcontroller.Memory;
 import de.darkress.pic16f84sim.microcontroller.ProgramCounter;
 
@@ -18,8 +17,6 @@ public class Rlf extends FileRegisterCommandUtils implements Command
     @Override
     public void execute()
     {
-        ProgramCounter.incPC();
-        Cycles.incCycles();
         int register = Memory.getRegister(address);
         int newCarry = register >>7;
         int oldCarry = Memory.getCarryBit();
@@ -33,5 +30,6 @@ public class Rlf extends FileRegisterCommandUtils implements Command
         }
 
         writeToDestination(destinationBit, address, register);
+        ProgramCounter.incPC();
     }
 }

de/darkress/pic16f84sim/commands/Rrf.java

@@ -1,6 +1,5 @@
 package de.darkress.pic16f84sim.commands;
 
-import de.darkress.pic16f84sim.microcontroller.Cycles;
 import de.darkress.pic16f84sim.microcontroller.Memory;
 import de.darkress.pic16f84sim.microcontroller.ProgramCounter;
 
@@ -18,8 +17,6 @@ public class Rrf extends FileRegisterCommandUtils implements Command
     @Override
     public void execute()
     {
-        ProgramCounter.incPC();
-        Cycles.incCycles();
         int register = Memory.getRegister(address);
         int newCarry = register & 0x01;
         int oldCarry = Memory.getCarryBit();
@@ -33,5 +30,6 @@ public class Rrf extends FileRegisterCommandUtils implements Command
         }
 
         writeToDestination(destinationBit, address, register);
+        ProgramCounter.incPC();
     }
 }

de/darkress/pic16f84sim/commands/Sublw.java

@@ -1,6 +1,5 @@
 package de.darkress.pic16f84sim.commands;
 
-import de.darkress.pic16f84sim.microcontroller.Cycles;
 import de.darkress.pic16f84sim.microcontroller.Memory;
 import de.darkress.pic16f84sim.microcontroller.ProgramCounter;
 
@@ -36,8 +35,6 @@ public class Sublw extends LiteralCommandUtils implements Command
     @Override
     public void execute()
     {
-        ProgramCounter.incPC();
-        Cycles.incCycles();
         int result = literal - Memory.workingRegister + 256;
 
         checkZeroBit(result);
@@ -45,5 +42,6 @@ public class Sublw extends LiteralCommandUtils implements Command
         checkDigitCarryBit(literal);
 
         Memory.workingRegister = result % 256;
+        ProgramCounter.incPC();
     }
 }

de/darkress/pic16f84sim/commands/Subwf.java

@@ -1,6 +1,5 @@
 package de.darkress.pic16f84sim.commands;
 
-import de.darkress.pic16f84sim.microcontroller.Cycles;
 import de.darkress.pic16f84sim.microcontroller.Memory;
 import de.darkress.pic16f84sim.microcontroller.ProgramCounter;
 
@@ -38,8 +37,6 @@ public class Subwf extends FileRegisterCommandUtils implements Command
     @Override
     public void execute()
     {
-        ProgramCounter.incPC();
-        Cycles.incCycles();
         int result = Memory.getRegister(address) - Memory.workingRegister + 256;
 
         checkZeroBit(result);
@@ -47,5 +44,6 @@ public class Subwf extends FileRegisterCommandUtils implements Command
         checkDigitCarryBit(Memory.getRegister(address));
 
         writeToDestination(destinationBit, address, result % 256);
+        ProgramCounter.incPC();
     }
 }

de/darkress/pic16f84sim/commands/Swapf.java

@@ -1,6 +1,5 @@
 package de.darkress.pic16f84sim.commands;
 
-import de.darkress.pic16f84sim.microcontroller.Cycles;
 import de.darkress.pic16f84sim.microcontroller.Memory;
 import de.darkress.pic16f84sim.microcontroller.ProgramCounter;
 
@@ -18,12 +17,11 @@ public class Swapf extends FileRegisterCommandUtils implements Command
     @Override
     public void execute()
     {
-        ProgramCounter.incPC();
-        Cycles.incCycles();
         int result = (Memory.getRegister(address) <<4) & 0xF0;
         int tmp = Memory.getRegister(address) >>4;
         result += tmp;
 
         writeToDestination(destinationBit, address, result);
+        ProgramCounter.incPC();
     }
 }

de/darkress/pic16f84sim/commands/Xorlw.java

@@ -1,6 +1,5 @@
 package de.darkress.pic16f84sim.commands;
 
-import de.darkress.pic16f84sim.microcontroller.Cycles;
 import de.darkress.pic16f84sim.microcontroller.Memory;
 import de.darkress.pic16f84sim.microcontroller.ProgramCounter;
 
@@ -15,12 +14,11 @@ public class Xorlw extends LiteralCommandUtils implements Command
     @Override
     public void execute()
     {
-        ProgramCounter.incPC();
-        Cycles.incCycles();
         int result = literal ^ Memory.workingRegister;
 
         checkZeroBit(result);
 
         Memory.workingRegister = result % 256;
+        ProgramCounter.incPC();
     }
 }

de/darkress/pic16f84sim/commands/Xorwf.java

@@ -1,6 +1,5 @@
 package de.darkress.pic16f84sim.commands;
 
-import de.darkress.pic16f84sim.microcontroller.Cycles;
 import de.darkress.pic16f84sim.microcontroller.Memory;
 import de.darkress.pic16f84sim.microcontroller.ProgramCounter;
 
@@ -18,12 +17,11 @@ public class Xorwf extends FileRegisterCommandUtils implements Command
     @Override
     public void execute()
     {
-        ProgramCounter.incPC();
-        Cycles.incCycles();
         int result = Memory.getRegister(address) ^ Memory.workingRegister;
 
         checkZeroBit(result);
 
         writeToDestination(destinationBit, address, result);
+        ProgramCounter.incPC();
     }
 }

de/darkress/pic16f84sim/microcontroller/Cycles.java

@@ -1,15 +0,0 @@
-package de.darkress.pic16f84sim.microcontroller;
-
-public class Cycles {
-    private static int cycles = 0;
-
-    public static void incCycles()
-    {
-        Timer.increaseTimer();
-        cycles++;
-    }
-
-    public static int getCycles() {
-        return cycles;
-    }
-}

de/darkress/pic16f84sim/microcontroller/Memory.java

@@ -4,13 +4,6 @@ import java.util.Arrays;
 
 public class Memory
 {
-    public static void initMemory()
-    {
-        Memory.memory[0x81] = 0xFF; //Option
-        Memory.memory[0x85] = 0x1F; //TrisA
-        Memory.memory[0x86] = 0xFF; //TrisB
-        Memory.setRegister(0x03, 0x18); //Status
-    }
     private static final int MEMORY_SIZE = 0xFF;  //Addressable Memory
     public static int workingRegister = 0;
     private static final int[] memory = new int[MEMORY_SIZE];
@@ -54,19 +47,12 @@ public class Memory
             setDataFromIndirectAddress(indirectAddress, data);
             return;
         }
-        if(address == 0x01) //Reset PrescalerCounter if change on Option or Timer Register
-        {
-            Timer.resetTimeToTimerIncrease();
-            Timer.setCyclesToTimerIncrease(Timer.getCyclesToTimerIncrease() - 1); //Decrease by one to account for
-            // this command execution
-        }
         if((Arrays.stream(bank0UniqueSpecialRegister).anyMatch(x -> x == address)) && getRegisterBank() == 0)
         {
             memory[address] = data;
             return;
         }
-        if((Arrays.stream(bank0UniqueSpecialRegister).anyMatch(x -> x == address)) && getRegisterBank() == 1) //bank0
-            // because of 7 Bit address
+        if((Arrays.stream(bank1UniqueSpecialRegister).anyMatch(x -> x == address)) && getRegisterBank() == 1)
         {
             memory[address + 128] = data;
             return;
@@ -86,12 +72,6 @@ public class Memory
 
     private static void setDataFromIndirectAddress(int address, int data)
     {
-        if(address == 0x81 || address == 0x01) //Reset PrescalerCounter if change on Option or Timer Register
-        {
-            Timer.resetTimeToTimerIncrease();
-            Timer.setCyclesToTimerIncrease(Timer.getCyclesToTimerIncrease() - 1); //Decrease by one to account for
-            // this command execution
-        }
         if((Arrays.stream(bank0UniqueSpecialRegister).anyMatch(x -> x == address)) || (Arrays.stream(bank1UniqueSpecialRegister).anyMatch(x -> x == address)))
         {
             memory[address] = data;
@@ -99,6 +79,10 @@ public class Memory
         }
         memory[address % 128] = data; // else: Registers.Registers which are mapped
         memory[address % 128 + 128] = data; //Ensure data is written to both banks to simulate mapping
+        if(address == 0x2) //Check if PCL is destination
+        {
+            ProgramCounter.loadPc();
+        }
     }
 
     private static int getRegisterBank()
@@ -137,21 +121,6 @@ public class Memory
         memory[0x8A] = data & 0x1F;
     }
 
-    public static int getOption()
-    {
-        return memory[0x81];
-    }
-
-    public static int getTimer()
-    {
-        return memory[0x01];
-    }
-
-    public static void setTimer(int data)
-    {
-        memory[0x01] = data;
-    }
-
     public static boolean getZeroBit()
     {
         return (memory[0x03] & 0x04) == 0x04;

de/darkress/pic16f84sim/microcontroller/Timer.java

@@ -1,66 +0,0 @@
-package de.darkress.pic16f84sim.microcontroller;
-
-public class Timer
-{
-    public static int getCyclesToTimerIncrease()
-    {
-        return cyclesToTimerIncrease;
-    }
-
-    public static void setCyclesToTimerIncrease(int cyclesToTimerIncrease)
-    {
-        Timer.cyclesToTimerIncrease = cyclesToTimerIncrease;
-    }
-
-    private static int cyclesToTimerIncrease = 1;
-
-    private static boolean timerEnabled()
-    {
-        return (Memory.getOption() & 0x20) != 0x20;
-    }
-
-    public static void resetTimeToTimerIncrease()
-    {
-        cyclesToTimerIncrease = getPrescalerFactor();
-    }
-
-    private static boolean getPrescalerAsssignment() {
-        return (Memory.getOption() & 0x08) == 0x08;
-    }
-
-    private static int getPrescalerFactor() {
-        final int MULTIPLIER = 2;
-        int prescalerPower = Memory.getOption() & 0x07;
-        int prescaler = (int)Math.pow(2, prescalerPower);
-        if(!getPrescalerAsssignment())
-        {
-            return prescaler * MULTIPLIER;
-        }
-        return prescaler;
-    }
-
-    private static void increaseTimerRegister()
-    {
-        int timerRegister = Memory.getRegister(0x01);
-        timerRegister = (timerRegister + 1) % 256;
-        if(timerRegister == 0) //check for timer Overflow  --> interrupt
-        {
-            System.out.println("Timer Overflow");
-        }
-        Memory.setTimer(timerRegister);
-    }
-
-    public static void increaseTimer()
-    {
-        if(!timerEnabled())
-        {
-            return;
-        }
-        cyclesToTimerIncrease--;
-        if(cyclesToTimerIncrease == 0)
-        {
-            resetTimeToTimerIncrease();
-            increaseTimerRegister();
-        }
-    }
-}

de/darkress/pic16f84sim/parser/Parser.java

@@ -1,8 +1,5 @@
 package de.darkress.pic16f84sim.parser;
 
-import de.darkress.pic16f84sim.commands.Command;
-import de.darkress.pic16f84sim.decoder.CommandDecoder;
-
 import java.io.File;
 import java.io.FileNotFoundException;
 import java.util.ArrayList;
@@ -11,9 +8,9 @@ import java.util.Scanner;
 
 public class Parser
 {
-    public static Command[] parser(String filePath)
+    public static List<Integer> parser(String filePath)
     {
-        ArrayList<String> instructions = new ArrayList<>();
+        ArrayList<Integer> program = new ArrayList<>();
         try {
             File file = new File(filePath);
             Scanner scanner = new Scanner(file);
@@ -22,10 +19,8 @@ public class Parser
                 if(!data.startsWith(" "))
                 {
                     String commandAndParametersString = "0x" + data.split(" ")[1];
-                    String instructionAddress = "0x" + data.split(" ")[0];
-                    //int commandAndParameters = Integer.decode(commandAndParametersString);
-                    //program.add(commandAndParameters);
-                    instructions.add(instructionAddress + ", " + commandAndParametersString);
+                    int commandAndParameters = Integer.decode(commandAndParametersString);
+                    program.add(commandAndParameters);
                 }
             }
             scanner.close();
@@ -33,16 +28,6 @@ public class Parser
             System.out.println("An error occurred.");
             e.printStackTrace();
         }
-
-        Command[] program = new Command[1024];
-
-        for(int i = 0; i < instructions.size(); i++)
-        {
-            int instructionAddress = Integer.decode(instructions.get(i).split(", ")[0]);
-            int instruction = Integer.decode(instructions.get(i).split(", ")[1]);
-            program[instructionAddress] = CommandDecoder.decode(instruction);
-        }
-
         return program;
     }
 }