Closed ghost closed 1 year ago
Hello!
Thanks for making out an pull request. However after a review we marked this pull request as unnecessary and inefficient.
For example: you have changed theswitch
statement which was used to check input with an very long if
condition. It is a bad practice to use a if
statement on long conditions instead of switch
. A switch
statement uses less memory in the situation.
We also see that you rename variable names which is also not needed. Everything is already self-explaining.
You have also accomplished doing inefficient functionalities. Like adding arguments to a function, as these parameter you defined were already drawn through the globals
.
So basically we won't be pulling this fork however thanks for your efforts. I appreciated your interest in ProxyBeast <3
3 feb 2023
) -> https://github.com/Z3NTL3/ProxyBeast/issues/5
Changes