Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(5053)

Issue 1868043: code review 1868043: syscall: On Windows, Errstr FormatMessage has no values...

Can't Edit
Can't Publish+Mail
Start Review
Created:
13 years, 9 months ago by peterGo
Modified:
13 years, 9 months ago
Reviewers:
brainman
CC:
rsc, brainman, golang-dev
Visibility:
Public.

Description

syscall: On Windows, Errstr FormatMessage has no values to insert. For the Windows version of syscall Errstr, set the FORMAT_MESSAGE_IGNORE_INSERTS value of the FormatMessage Flags argument when there are no values to insert.

Patch Set 1 #

Patch Set 2 : code review 1868043: syscall: On Windows, Errstr FormatMessage has no values... #

Unified diffs Side-by-side diffs Delta from patch set Stats (+2 lines, -1 line) Patch
M src/pkg/syscall/syscall_windows.go View 1 1 chunk +2 lines, -1 line 0 comments Download

Messages

Total messages: 4
peterGo
The FormatMessage Windows API supports dynamically formatted messages, where a variable list of arguments is ...
13 years, 9 months ago (2010-07-21 05:00:05 UTC) #1
peterGo
Hello rsc (cc: golang-dev@googlegroups.com), I'd like you to review this change.
13 years, 9 months ago (2010-07-21 05:01:26 UTC) #2
brainman
LGTM
13 years, 9 months ago (2010-07-21 09:13:33 UTC) #3
rsc
13 years, 9 months ago (2010-07-21 16:40:20 UTC) #4
*** Submitted as http://code.google.com/p/go/source/detail?r=79d86caba18a ***

syscall: On Windows, Errstr FormatMessage has no values to insert.

For the Windows version of syscall Errstr, set the
FORMAT_MESSAGE_IGNORE_INSERTS value of the FormatMessage
Flags argument when there are no values to insert.

R=rsc, brainman
CC=golang-dev
http://codereview.appspot.com/1868043

Committer: Russ Cox <rsc@golang.org>
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld f62528b