Skip to content
This repository has been archived by the owner on Aug 2, 2022. It is now read-only.

Fix for empty ("") appbase config default value #9149

Merged
merged 1 commit into from
Jun 2, 2020

Conversation

heifner
Copy link
Contributor

@heifner heifner commented May 29, 2020

Change Description

  • Specifying an empty "" default value would fail when trying to create a new config.ini file.
    • Example:
void new_plugin::set_program_options(options_description& cli, options_description& cfg) {
   auto op = cfg.add_options();
   op("option-one", bpo::value<std::string>()->default_value(""), "Option");
}

Consensus Changes

  • Consensus Changes

API Changes

  • API Changes

Documentation Additions

  • Documentation Additions

@heifner heifner merged commit 2c7be2d into develop Jun 2, 2020
@heifner heifner deleted the app-empty-default-dev branch June 2, 2020 13:57
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants