Skip to content

Make several symbols related to library formats private#9852

Merged
WalterBright merged 1 commit intodlang:masterfrom
andralex:lib-private
May 23, 2019
Merged

Make several symbols related to library formats private#9852
WalterBright merged 1 commit intodlang:masterfrom
andralex:lib-private

Conversation

@andralex
Copy link
Member

This is investigating how many of the symbols in lib-related code could be encapsulated as private, vs. repackaged per #9844. I only built on Linux so I expect errors specific to Windows/Mac.

@dlang-bot
Copy link
Contributor

Thanks for your pull request, @andralex!

Bugzilla references

Your PR doesn't reference any Bugzilla issue.

If your PR contains non-trivial changes, please reference a Bugzilla issue or create a manual changelog.

Testing this PR locally

If you don't have a local development environment setup, you can use Digger to test this PR:

dub fetch digger
dub run digger -- build "master + dmd#9852"

Copy link
Contributor

@wilzbach wilzbach left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What's wrong with #9844?

@andralex
Copy link
Member Author

@wilzbach #9844 was not accepted by the project leader.

@andralex andralex mentioned this pull request May 23, 2019
@WalterBright
Copy link
Member

What's wrong with #9844?

private is the first step to better encapsulation, not package.

@WalterBright WalterBright merged commit 61bd4d1 into dlang:master May 23, 2019
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.

4 participants