Join GitHub today
GitHub is home to over 31 million developers working together to host and review code, manage projects, and build software together.
Sign upprivate type in public interface rules do not prevent exposure of some types #23621
Comments
rust-highfive
added
the
I-nominated
label
Mar 22, 2015
nrc
referenced this issue
Mar 22, 2015
Closed
Private type in exported type signature when it shouldn't #23585
nrc
added
A-resolve
A-visibility
labels
Mar 22, 2015
This comment has been minimized.
This comment has been minimized.
|
The RFC did not try to prevent this case |
This comment has been minimized.
This comment has been minimized.
|
As discussed in the RFC, we try to prevent private items in public signatures, not types which are not visible, it was decided that the latter was too difficult. So, this is not a bug. Closing. |
nrc
closed this
Mar 24, 2015
nrc
removed
the
I-nominated
label
Mar 24, 2015
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
nrc commentedMar 22, 2015
e.g.,
From outside
B,foois visible, butXis not.The fix is discussed here and in the following comments.
I guess this is technically a backwards incompatible change, but I would regard any example of it as a bug, so I would not feel bad fixing this post 1.0 (also the fix is really easy if we do break your code - just add
pub). Still, nominating triage: I-nominated(example by @tomaka from #23585)