Re: Review Request 45726: Fixed a bug in the `flags::parse` function and added a test case.

2016-04-05 Thread Jie Yu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45726/#review127151 --- Ship it! Ship It! - Jie Yu On April 5, 2016, 5:33 p.m., Mich

Re: Review Request 45726: Fixed a bug in the `flags::parse` function and added a test case.

2016-04-05 Thread Michael Park
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45726/ --- (Updated April 5, 2016, 5:33 p.m.) Review request for mesos and Jie Yu. Chang

Re: Review Request 45726: Fixed a bug in the `flags::parse` function and added a test case.

2016-04-04 Thread Mesos ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45726/#review127032 --- Patch looks great! Reviews applied: [45726] Passed command: expo

Re: Review Request 45726: Fixed a bug in the `flags::parse` function and added a test case.

2016-04-04 Thread Jie Yu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45726/#review127023 --- Can you also add a test case for the empty string case? e.g., `--n

Re: Review Request 45726: Fixed a bug in the `flags::parse` function and added a test case.

2016-04-04 Thread Jie Yu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45726/#review127021 --- Ship it! Ship It! - Jie Yu On April 5, 2016, 2:57 a.m., Mich

Review Request 45726: Fixed a bug in the `flags::parse` function and added a test case.

2016-04-04 Thread Michael Park
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45726/ --- Review request for mesos and Jie Yu. Bugs: MESOS-5114 https://issues.apache