I suppose the developers did not intend to directly access internal Impl class 
but was rather lacking proper interface API. In fact, throughout the 
repository, I only found one place that uses `RandGenerator<cpu,` GType>::Impl` 
directly, and therefore I think it's not a big overhead to refactor that one 
line of code. Regarding how to separating Impl class from the interface, you 
might find this reference helpful: https://cpppatterns.com/patterns/pimpl.html

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

Reply via email to