I like the feature, but I have two comments:

1. I am not sure how I feel about `@` as a prefix.  Since this is already used 
to designate a repo name, I worry this could be confusing for users.  I wonder 
if it would be clearer if the notation were: `MYNEWT_VAL(setting_name)`.  I am 
really not sure, though... if you think `@` is fine then it is probably OK.

2. Would you mind renaming `ResolveValues()` to something less generic?  Maybe 
`ResolveReferences()`?

[ Full content available at: https://github.com/apache/mynewt-newt/pull/201 ]
This message was relayed via gitbox.apache.org for [email protected]

Reply via email to