http://llvm.org/bugs/show_bug.cgi?id=21937

            Bug ID: 21937
           Summary: src/locale.cpp's time formatting: couldn't this just
                    use nl_langinfo_l()?
           Product: libc++
           Version: unspecified
          Hardware: PC
                OS: Linux
            Status: NEW
          Severity: enhancement
          Priority: P
         Component: All Bugs
          Assignee: [email protected]
          Reporter: [email protected]
                CC: [email protected], [email protected]
    Classification: Unclassified

In __time_get_storage<wchar_t>::__analyze() we seem to 'reverse engineer' the
internal format string used by strftime(). We format an artifical date/time
string and then parse it again to obtain the format string. The same holds for
init(), where we iteratively call strftime() to obtain the names of the days
and months.

Though impressive, such a construct is actually not needed. Information like
this can be extracted from the C library directly using nl_langinfo() or
nl_langinfo_l().

Just filing this bug to make sure this won't get lost.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
_______________________________________________
LLVMbugs mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/llvmbugs

Reply via email to