search

ethercat-rs / ethercat

Rust wrapper for the IgH EtherCAT master
Apache License 2.0
58 stars 18 forks source link

Small refactoring #6

Closed flosse closed 4 years ago

flosse commented 4 years ago

@birkenfeld I couldn't find a difference between DomainHandle and DomainIndex so I simplified some lines by getting rid of DomainHandle.

birkenfeld commented 4 years ago

@flosse I think the original intent was to have DomainHandle be an opaque API type, while DomainIndex would be internal only. As you say, it might be moot to distinguish.

flosse commented 4 years ago

Thanks for your review!

birkenfeld commented 4 years ago

Good to go!