saleyn / erlexec

Execute and control OS processes from Erlang/OTP
https://hexdocs.pm/erlexec/readme.html
Other
525 stars 139 forks source link

Create CmdInfo objects directly in the map and fix one error from clang 11 #144

Closed chrta closed 3 years ago

chrta commented 3 years ago

Next round... (probably the last one for now)

I had a look at compiler warnings etc from clang 11 and fixed a few. My initial intention was to remove new(this)Class(); because of https://isocpp.org/wiki/faq/ctors#init-methods and use c++11 delegating constructor. But since CmdInfo is only used in a map, i instead use only the move constructor and removed the copy constructor (and also the default operator=()).

Summary: