Incorrect Java single quotes handling results in error when student submits correct java code

[PatTutor]

Hi

We love your plugin!

We found the following issue: When a student submits the following java code, there is a Java error message despite the student submitting correct code:

package ch.zhaw.ads;

class KgvServer implements CommandExecutor {
    public int kgv(int a, int b) {
        return 0;
    }
    
    @Override
    public String execute(String command) {
        // char c = '{'
        return "Hello" + '{';
    }
}

We already have fixed this and will add a pull request.

Thanks!
Pat

[trampgeek]

@PatTutor Thanks for reporting and for the pull request.

Your pull request looks good, thanks, but before I merge I'd like to check that I'm understanding properly. I can see that literal brace characters in the code will break the check that removes nested code, which could then lead to a subsequent public class being filtered out. But I'm not seeing how a Java error message would arise with the above example code. When I run the get_main_class function on that code it correctly returns false , as there's no main class. Was there additional code following the above with a main class in it?