Skip to content
This repository was archived by the owner on Jun 30, 2025. It is now read-only.

Conversation

anpol
Copy link
Contributor

@anpol anpol commented Aug 12, 2022

Calling FormatMessage without a FORMAT_MESSAGE_IGNORE_INSERTS flag could cause trouble, as explained in an article by Raymond Chen:

Calling FormatMessage without a FORMAT_MESSAGE_IGNORE_INSERTS flag could
cause trouble, as explained in an article by Raymond Chen:

* The importance of the FORMAT_MESSAGE_IGNORE_INSERTS flag
  https://devblogs.microsoft.com/oldnewthing/20071128-00/?p=24353
@codecov-commenter
Copy link

codecov-commenter commented Aug 12, 2022

Codecov Report

Merging #849 (79160d4) into master (278ed96) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #849   +/-   ##
=======================================
  Coverage   73.17%   73.17%           
=======================================
  Files          17       17           
  Lines        3277     3277           
=======================================
  Hits         2398     2398           
  Misses        879      879           
Impacted Files Coverage Δ
src/glog/logging.h.in 79.45% <ø> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@sergiud sergiud added this to the 0.7 milestone Aug 12, 2022
@sergiud
Copy link
Contributor

sergiud commented Aug 12, 2022

Sounds reasonable. Thanks!

@sergiud sergiud merged commit a1b6164 into google:master Aug 12, 2022
@anpol anpol deleted the format-ignore-inserts branch August 13, 2022 08:08
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants