Keyboard Shortcuts

File
u :up to issue
m :publish + mail comments
M :edit review message
j / k :jump to file after / before current file
J / K :jump to next file with a comment after / before current file
Side-by-side diff
i :toggle intra-line diffs
e :expand all comments
c :collapse all comments
s :toggle showing all comments
n / p :next / previous diff chunk or comment
N / P :next / previous comment
<Up> / <Down> :next / previous line
<Enter> :respond to / edit current comment
d :mark current comment as done
Issue
u :up to list of issues
m :publish + mail comments
j / k :jump to patch after / before current patch
o / <Enter> :open current patch in side-by-side view
i :open current patch in unified diff view
Issue List
j / k :jump to issue after / before current issue
o / <Enter> :open current issue
# : close issue
Comment/message editing
<Ctrl> + s or <Ctrl> + Enter :save comment
<Esc> :cancel edit
Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(275)
Issues Repositories Search
Open Issues | Closed Issues | All Issues | Sign in with your Google Account to create issues and add comments

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

Can't Edit
Can't Publish+Mail
Start Review
Created:
15 years, 5 months ago by peterGo
Modified:
15 years, 5 months ago
Reviewers:
brainman
CC:
rsc, brainman, golang-dev
Visibility:
Public.
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... #

Created: 15 years, 5 months ago
Download [raw] [tar.bz2]
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
Total messages: 4
|
peterGo
The FormatMessage Windows API supports dynamically formatted messages, where a variable list of arguments is ...
15 years, 5 months ago (2010年07月21日 05:00:05 UTC) #1
The FormatMessage Windows API supports dynamically formatted messages, where a
variable list of arguments is used to insert values in the formatted message.
For example,
ERROR_WRONG_DISK = 34 "The wrong diskette is in the drive. Insert %2 (Volume
Serial Number: %3) into drive %1."
The formatting options are given in the Flags argument. If the message to be
formatted has dynamic inserts, but there are no values available to be inserted,
the FORMAT_MESSAGE_IGNORE_INSERTS value of Flags must be set. It's not
sufficient to just set the pointer to the values to be inserted to null; it
fails with ERROR_INVALID_PARAMETER, 87, "The parameter is incorrect."
Sign in to reply to this message.
peterGo
Hello rsc (cc: golang-dev@googlegroups.com), I'd like you to review this change.
15 years, 5 months ago (2010年07月21日 05:01:26 UTC) #2
Hello rsc (cc: golang-dev@googlegroups.com),
I'd like you to review this change.
Sign in to reply to this message.
brainman
LGTM
15 years, 5 months ago (2010年07月21日 09:13:33 UTC) #3
LGTM
Sign in to reply to this message.
rsc
*** Submitted as http://code.google.com/p/go/source/detail?r=79d86caba18a *** syscall: On Windows, Errstr FormatMessage has no values to insert. ...
15 years, 5 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

AltStyle によって変換されたページ (->オリジナル) /