Menu

#2272 [SciTE] CurrentTime might get ansi string in Windows

Bug
closed-fixed
nobody
5
2021-09-22
2021-08-05
twu2
No

In SciTEWinBar.cxx SciTEWin::SetFileProperties(), use GetTimeFormatA()/GetDateFormatA() might get non-English ansi string.
We shoud use GetTimeFormatW()/GetDateFormatW() to get unicode string.

1 Attachments

Discussion

  • Neil Hodgson

    Neil Hodgson - 2021-08-05
    • labels: SciTE, Windows --> SciTE, Windows, win32, locale
    • status: open --> open-fixed
     
  • Neil Hodgson

    Neil Hodgson - 2021-08-05

    All the regions I tried use only ASCII punctuation and digits for time or short dates so there was no effect.

    Change was committed as [0a42b7] with the c_str calls removed as they aren't needed.

    Credited to "tommywu" which can be changed if you'd prefer a different name or formatting.

     

    Related

    Commit: [0a42b7]

  • Neil Hodgson

    Neil Hodgson - 2021-09-22
    • status: open-fixed --> closed-fixed
     

Log in to post a comment.

MongoDB Logo MongoDB