Skip to content

JSON output

Everton Constantino requested to merge everton.constantino/windowsperf:json2 into main
  • Include JSON output class and adapt PrettyTable to abstract output type with TableOutput.
  • Include -q option to handle quiet outputting.
  • Include -json option to output JSON instead of PrettyTable.
  • Include --output to set filename where the output will be written to.
  • Start supporting the option to handle wide strings along with wide strings output.

Handles WPERF-62/63/64/65

@PrzemekWirkus This is still a work in progress as I need to change some table names and include the schema but I believe the architecture is more or less finished. I had to tackle everything at once as I tried to handle things separately I started to find some flaws with my design and ended up having to do a complete flow before advancing.

The JSON class might look a bit too generic but this is intended as I will expand this further along with PrettyTable but I wanted to get the ground work done before going any further.

This is still marked as Draft, I rushed a little bit to get your opinion before the 15th hence the early merge request.

I had to move PrettyTable implementation to a single file for C++-ish reasons that I didn't want to fight against at the time.

If you have some time to take a quick look and give your opinion it would be nice.

Merge request reports