search

eao197 / procyy

A reincarnation of procxx library: a simple process management library for C++ on UNIX platforms.
MIT License
10 stars 0 forks source link

process move constructor implicitly deleted #2

Closed bmcdanie closed 4 years ago

bmcdanie commented 4 years ago

process move constructor is marked explicitly defaulted, but is deleted - probably due to problems with members' move constructors. I refactored my code around it, so I'm not waiting for a fix, just FYI. this problem also occurs in the original procxx.

eao197 commented 4 years ago

Thanks for reporting this! We have never used process with move operations so this issue was hidden for a long time.

I've changed the implementation of process class. Now it uses PImpl idiom, which makes move constructor/operator available for process instances.

bmcdanie commented 4 years ago

excellent, thank you!