Apollo: issue about cyber/time/time.cc#100 && #104

Created on 12 Sep 2019  ·  5Comments  ·  Source: ApolloAuto/apollo

may I use << setw(9) << setfill('0') after ss << std::string(date_time) << "." ?

Cyber Question
Source
picture zhuangli1987
🚀1 😄1 👍1

Most helpful comment

nice catch

picture lianglia-apollo on 12 Sep 2019
🎉1 👍1

All 5 comments

nice catch

lianglia-apollo picture lianglia-apollo on 12 Sep 2019
🎉1 👍1

Fixed with #9664

fengqikai1414 picture fengqikai1414 on 12 Sep 2019
👍1

@zhuangli1987 @lianglia-apollo thank you for your continued support towards the Apollo platform, and we encourage a lot more PRs and issues from you! :) Closing this for now as it's resolved!

natashadsouza picture natashadsouza on 12 Sep 2019
👍1

Fixed with #9664

Hi, fengqikai, I think the code in this patch should be better if we use ss << "." << std::setw(9) << std::setfill('0') . at line 97.

zhuangli1987 picture zhuangli1987 on 12 Sep 2019

@zhuangli1987 you are right. I will fix soon.

fengqikai1414 picture fengqikai1414 on 16 Sep 2019
Was this page helpful?
0 / 5 - 0 ratings

Related issues

freeclouds picture freeclouds  ·  3Comments
chilihua picture chilihua  ·  3Comments
lyletbjohnson picture lyletbjohnson  ·  3Comments
xinwf picture xinwf  ·  3Comments
ahuer2435 picture ahuer2435  ·  3Comments