AbstractJavaVirtualMachine.getCommandLine() is currently stubbed out, can't retrieve CLI parameters for Pre-Unified log.

[Bluetopia]

Describe the bug
The getCommandLine() method has been stubbed out with a TODO. See

gctoolkit/api/src/main/java/com/microsoft/gctoolkit/jvm/AbstractJavaVirtualMachine.java

Line 84 in 1476fc1

public String getCommandLine() {

Expected behavior
Expected this method, or child override in PreUnifiedJavaVirtualMachine to return the command line string from a Pre-Unified log file.

Additional context
There is a pending issue for building the Command Line args for Unified logging here: #263 which states that retrieving the command line is supported for Pre-Unified log files. However, the AbstractJavaVirtualMachine.getCommandLine() method is not overridden in PreUnifiedJavaVirtualMachine so the Abstract method is used instead.

Not sure if this was stubbed out while waiting for issue 263 or an oversight.

[karianna]

@kcpeppe & @dsgrieve for comment

[kcpeppe]

This is behaviour that we had in Censum but didn't get transferred to GCToolKit. It is a lower priority item for me ATM.