pyreportjasper icon indicating copy to clipboard operation
pyreportjasper copied to clipboard

report.py lines 61 and 63 seem to have a problem

Open massimiliano-della-rovere opened this issue 6 months ago • 2 comments

I'm not sure if this is a bug, but seems strange code.

The file is report.py, class Report, method __init__, inside the if not jpype.isJVMStarted() there are 2 ifs, one in line 61 and one in line 63. The combined logic of their evaluation seems... strange to me: if the condition of the if at line 61 is True, then the classpath list is updated, but the condition of the if at line 63 will automatically eval to False, so the updated classpath won't be used and will be thrashed since it's not used anywhere else in the __init__ method.

Am I missing something?

immagine