Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Remove inPlace #10

Merged
merged 1 commit into from
Aug 4, 2016
Merged

Remove inPlace #10

merged 1 commit into from
Aug 4, 2016

Commits on Aug 4, 2016

  1. Remove inPlace

    It looks like we don't actually use the inPlace argument to the
    PropertyList add and set methods anywhere in the stack besides the
    daf_base unit tests. I think it was originally put in to accommodate the
    addition of COMMENT and HISTORY items that might want to be placed at
    the end of the PropertyList instead of grouped together, but the
    implementation actually still groups all of these items together, just
    at the end of the list instead of where they were.
    ktlim authored and timj committed Aug 4, 2016
    Configuration menu
    Copy the full SHA
    4a41107 View commit details
    Browse the repository at this point in the history