Skip to content

Exported Config.Data & Merge()#1

Open
jswank wants to merge 1 commit intostathat:masterfrom
jswank:master
Open

Exported Config.Data & Merge()#1
jswank wants to merge 1 commit intostathat:masterfrom
jswank:master

Conversation

@jswank
Copy link

@jswank jswank commented May 22, 2012

Renamed Config.data to Config.Data and merge() to Merge().

I found these minor changes to be convenient wrt/ setting defaults for optional configuration directives or overriding config directives based on other input.

    defaults := map[string]interface{}{
        "foo": "on",
        "bar": 1,
        "baz": true,
    }

    config := &jconfig.Config{}
    config.Data = defaults

    config.StringMerge(`{"foo": "off"}`)

    fmt.Printf("optional config baz: %t\n", config.GetBool("baz"))

export merge() function (rename merge() to Merge())
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant