speedb icon indicating copy to clipboard operation
speedb copied to clipboard

Use OptionsLogFormatter for Options::Dump

Open mrambacher opened this issue 1 year ago • 6 comments

This PR replaces the code in the Options::Dump method with code using the Options infrastructure. A new "SerializePrintableOptions" method was added to the Configurable class which allows the values not serializable by the options to be included in the output. A ConfigOptions::Depth::KPrintable was added to tell the classes to invoked this method as part of the serialization process.

The Options::Dump methods now use the this infrastructure to generate their output. The code in the GetPrintableOptions methods of various was either deleted (because the values were serialized as Options) or moved into the SerializePrintableOptions method.

mrambacher avatar Oct 15 '23 16:10 mrambacher

@ofriedma after #651

Guyme avatar Oct 30 '23 13:10 Guyme

@mrambacher can you please solve conflicts so I can review it?

ofriedma avatar Nov 15 '23 15:11 ofriedma

This PR fixes #692

mrambacher avatar Nov 19 '23 21:11 mrambacher

lgtm

ofriedma avatar Nov 26 '23 14:11 ofriedma

I manually tested the ARM compile, it worked fine

ofriedma avatar Nov 27 '23 14:11 ofriedma

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.
4 out of 9 committers have signed the CLA.

:white_check_mark: Yuval-Ariel
:white_check_mark: ofriedma
:white_check_mark: udi-speedb
:white_check_mark: maxb-io
:x: mrambacher
:x: git-hulk
:x: GitHub Runner Bot
:x: zaidoon1
:x: ayulas


GitHub Runner Bot seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.
You have signed the CLA already but the status is still pending? Let us recheck it.

CLAassistant avatar Feb 01 '24 08:02 CLAassistant