User Details
- User Since
- Mar 4 2014, 12:37 PM (499 w, 5 d)
May 31 2023
May 25 2023
Apr 1 2023
Mar 31 2023
Comments within. Generally LGTM
Mar 15 2023
Mar 14 2023
Mar 13 2023
Mar 12 2023
Mar 1 2023
Feb 25 2023
Feb 22 2023
Update comments per Siva's feedback.
Feb 21 2023
Feb 20 2023
Add missing newline.
Feb 19 2023
Feb 15 2023
Jan 24 2023
Jan 5 2023
Jan 1 2023
Dec 22 2022
I don't think landing this without it having any way of displaying a different time than gmtime is a good idea. Happily, I started working on the TZ patch again, so this might be good to land early in the year.
Nov 11 2022
Did you intend to land this without review?
Nov 7 2022
Ate these test changes enough to catch this? It seems a bit magical with the numbers. Should there be an explicit test that tests matching more than one field?
Nov 3 2022
I'm marking this as accepted - only the language annotation on the code blocks remains, but I think you'll have to just try something and see how the github MD interpreter copes with it. Thanks!
Nov 2 2022
Nov 1 2022
I'm not sure that the repeated description of sysroot is necessary, but I'm not sure that it's not. I always worry about duplicated info.
Oct 26 2022
Reviewing from my phone, but LGTM
Oct 24 2022
Oct 23 2022
Can you please update the status doc at
https://libc.llvm.org/date_and_time.html ?
Oct 17 2022
Oct 15 2022
Oct 7 2022
Sep 23 2022
Sep 20 2022
Sep 19 2022
Sep 2 2022
Sep 1 2022
Fix topic sentence.
Aug 19 2022
Aug 17 2022
Aug 8 2022
Aug 7 2022
Aug 5 2022
Aug 4 2022
Aug 3 2022
I think this closes out the comments. I'll leave this another day before submitting. Thanks all!
We don't have span, or I'd be tempted to suggest using it for that code.
Jul 30 2022
Oops, I noticed that I didn't have clang-format installed. This
diff fixes one formatting error.
Added two tests to test code review comment that noticed that an unset
optional would become set on assignment. Then fixed the code for
that situation by factoring our storage into a consistent object.
Jul 26 2022
Took a quick review while I had a break this morning.
Jul 22 2022
One thing: the test builds should fail with this as this patch depends on my optional.h patch that I'm still reworking. (I'm trying to learn enough about c++ traits to implement the ones I need). However, Raman and I work together on Fridays on our patches and we wanted to get this up for review since it's a large one.