Skip to content
This repository has been archived by the owner on Dec 1, 2022. It is now read-only.

Add more datetime literal format. #589

Merged
merged 2 commits into from
Jul 26, 2021

Conversation

Shylock-Hg
Copy link
Contributor

No description provided.

@Shylock-Hg Shylock-Hg added the ready-for-testing PR: ready for the CI test label Jul 20, 2021
@Shylock-Hg Shylock-Hg requested a review from a team July 20, 2021 02:53
Comment on lines +54 to +58
std::istringstream ss2(str);
ss2 >> std::get_time(&tm, "%Y-%m-%d %H:%M:%S");
if (ss2.fail()) {
return Status::Error();
}
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Does it conflict with this pr?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Will

@Shylock-Hg Shylock-Hg requested review from Aiee and a team July 20, 2021 06:08
std::istringstream ss2(str);
ss2 >> std::get_time(&tm, "%Y-%m-%d %H:%M:%S");
if (ss2.fail()) {
return Status::Error();
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

add the message

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
ready-for-testing PR: ready for the CI test
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants