Remove IArguments

Whilst it was historically useful, now that the string bug has been
fixed we no longer have a use for it.

Author: SquidDev

README.md

@@ -13,18 +13,14 @@ This will inject CCTweaks into the class loader, adding modifications.
 ## Features
  - Custom computer timeout
  - Whitelist globals (such as debug)
- - Fix the binary strings (for fs, http, rednet and os.queueEvent)
- - TCP socket API (`socket`)
+ - TCP and websocket socket API (`socket`)
  - Compression API (`data`)
  - [Cobalt](https://github.com/SquidDev/Cobalt) VM (reentrant fork of LuaJ)
    - Custom termination handler
    - Several bugs fixed (any object error messages, string pattern matching, number format strings)
    - Run multiple computers at once
  - Additional VMs provided by [CCTweaks-Runtimes](https://github.com/SquidDev-CC/CCTweaks-Runtimes)
- - Return HTTP handle on failures
- - Allow getting headers from HTTP responses
  - API for adding custom APIs
- - Limit number of open file handles
  - Boot from custom `bios.lua` file
 
 ## Documentation

src/main/java/org/squiddev/cctweaks/api/lua/ArgumentDelegator.java

@@ -24,10 +24,6 @@ public static void setup(TransformationChain chain) {
 			"dan200.computercraft.core.computer.Computer",
 			"org.squiddev.cctweaks.lua.patch.Computer_Patch"
 		));
-		// Binary patches
-		chain.add(new BinaryMachine());
-		chain.add(new ClassMerger(TweaksLogger.instance, "dan200.computercraft.core.apis.PeripheralAPI", "org.squiddev.cctweaks.lua.patch.PeripheralAPI_Patch"));
-		chain.add(new ClassMerger(TweaksLogger.instance, "dan200.computercraft.shared.peripheral.modem.ModemPeripheral", "org.squiddev.cctweaks.lua.patch.ModemPeripheral_Patch"));
 	}
 
 	private static void addMulti(TransformationChain chain, ClassReplacer replacer) {

src/main/java/org/squiddev/cctweaks/api/lua/IArguments.java

@@ -2,7 +2,10 @@
 
 import dan200.computercraft.api.lua.ILuaContext;
 import dan200.computercraft.api.lua.LuaException;
-import org.squiddev.cctweaks.api.lua.*;
+import org.squiddev.cctweaks.api.lua.IExtendedComputerAccess;
+import org.squiddev.cctweaks.api.lua.ILuaAPI;
+import org.squiddev.cctweaks.api.lua.ILuaAPIFactory;
+import org.squiddev.cctweaks.api.lua.IMethodDescriptor;
 import org.squiddev.cctweaks.lua.Config;
 
 import javax.annotation.Nonnull;
@@ -18,7 +21,7 @@
 /**
  * Adds inflate/deflate APIs
  */
-public class DataAPI implements ILuaAPI, ILuaObjectWithArguments, ILuaAPIFactory, IMethodDescriptor {
+public class DataAPI implements ILuaAPI, ILuaAPIFactory, IMethodDescriptor {
 	@Override
 	public void startup() {
 	}
@@ -61,19 +64,6 @@ public Object[] callMethod(@Nonnull ILuaContext context, int method, @Nonnull Ob
 		return null;
 	}
 
-	@Override
-	public Object[] callMethod(@Nonnull ILuaContext context, int method, @Nonnull IArguments args) throws LuaException, InterruptedException {
-		switch (method) {
-			case 0:
-				return inflate(args.getStringBytes(0));
-
-			case 1:
-				return deflate(args.getStringBytes(0));
-		}
-
-		return null;
-	}
-
 	private Object[] inflate(byte[] data) throws LuaException {
 		if (data.length >= Config.APIs.Data.limit) throw new LuaException("Data is too long");
 

src/main/java/org/squiddev/cctweaks/api/lua/ILuaObjectWithArguments.java

@@ -169,7 +169,7 @@ public static String getPreBios() {
 		return factory.getPreBios();
 	}
 
-	private static class LuaAPI implements ILuaAPI, ILuaObjectWithArguments, IExtendedLuaObject, IMethodDescriptor {
+	private static class LuaAPI implements ILuaAPI, IExtendedLuaObject, IMethodDescriptor {
 		private final org.squiddev.cctweaks.api.lua.ILuaAPI api;
 		private final ILuaAPIFactory factory;
 
@@ -209,11 +209,6 @@ public Object[] callMethod(@Nonnull ILuaContext context, int method, @Nonnull Ob
 			return api.callMethod(context, method, args);
 		}
 
-		@Override
-		public Object[] callMethod(@Nonnull ILuaContext context, int method, @Nonnull IArguments arguments) throws LuaException, InterruptedException {
-			return ArgumentDelegator.delegateLuaObject(api, context, method, arguments);
-		}
-
 		@Nonnull
 		@Override
 		public Map<Object, Object> getAdditionalData() {