Add Into<u32> for Pid - Githubissues

jabedude / ofiles

Small library for associating processes and open files

BSD 3-Clause "New" or "Revised" License

9 stars 5 forks source link

Closed JojiiOfficial closed 4 years ago

JojiiOfficial commented 4 years ago

Fixes #2

jabedude commented 4 years ago

Thanks for the PR! I think it would be better to instead implement From<Pid> for u32 here, since it automatically provides Into.

JojiiOfficial commented 4 years ago

Makes sense. Here the changes :)

jabedude commented 4 years ago

Great, thanks! Can you squash your two commits into one please?

jabedude commented 4 years ago

Thank you :)