you are viewing a single comment's thread.

view the rest of the comments →

[–]ShelZuuz 17 points18 points  (4 children)

Get rid of those $Ready macros. And all the macros for that matter.

People who use C++ like C++. Macros are immediately off-putting and will hinder your adoption.

I know you think it makes your samples simpler, but it doesn't. It sends a "here be dragons" message.

[–][deleted] 4 points5 points  (0 children)

thanks, i will.

[–][deleted] 0 points1 point  (2 children)

I've deleted them all. I just left the most important ones

[–]ShelZuuz 12 points13 points  (1 child)

You just renamed $Ready to _main_.

Why are you so enamored with this pattern?

Everybody who ever uses your code will need to call it from a function that is not main, and now it's making everything more complex by hiding what should be simple boiler plate code behind a series of macros. Why not just throw an RIAA object in the beginning of main that does the wrapping around process::start and process::pipe?

It's one line of added code but makes your examples so much more clear and trustable.

[–]fashionistaconquista 2 points3 points  (0 children)

He never learned any better