Mir

Code review comment for lp:~kdub/mir/mirscreencast-limited-capture

Revision history for this message
Kevin DuBois (kdub) wrote :

> 56 + number_of_captures = atoi(optarg);
>
> Maybe replace with
> std::stoi(std::string(optarg));
>
> Then it will throw if it can't parse a number.

It should return 0 if it can't parse and then it would just exit. I don't mind fixing to print the help msg on a bad param though.

« Back to merge proposal