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

Revert "Hide most of non-public symbols by default" #1038

Merged
merged 1 commit into from
Sep 28, 2021

Conversation

jbeder
Copy link
Owner

@jbeder jbeder commented Sep 28, 2021

Reverts #984

Fixes #1036.

@jbeder jbeder merged commit 0d9dbcf into master Sep 28, 2021
pinotree added a commit to pinotree/yaml-cpp that referenced this pull request Oct 9, 2021
…" (jbeder#1038)"

The issue jbeder#1038 is very generic, and it does not provide any detail on
what the actual problems are. Since the hidden visibility changes were
correct, revert their revertion to make them available again.

This reverts commit 0d9dbcf.
jbeder added a commit that referenced this pull request Nov 19, 2021
jbeder pushed a commit that referenced this pull request Nov 23, 2021
…1064)

Partial revert of "Revert "Revert "Hide most of non-public symbols by default (#984)" (#1038)" (#1045)"

This reverts commit 0733aeb.
davemccann pushed a commit to davemccann/yaml-cpp that referenced this pull request Jul 30, 2023
…beder#1064)

Partial revert of "Revert "Revert "Hide most of non-public symbols by default (jbeder#984)" (jbeder#1038)" (jbeder#1045)"

This reverts commit 0733aeb.
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.

Latest change result in dll.h not found error
1 participant