Add new VS AddIn examples

Add new VS AddIn examples.


BUG=

[binji, 2012-10-15 22:15:46]

http://codereview.chromium.org/11150028/diff/1/visual_studio/NativeClientVSAd...
File
visual_studio/NativeClientVSAddIn/InstallerResources/examples/hello_nacl/hello_nacl.sln
(right):

http://codereview.chromium.org/11150028/diff/1/visual_studio/NativeClientVSAd...
visual_studio/NativeClientVSAddIn/InstallerResources/examples/hello_nacl/hello_nacl.sln:8:
Debug|NaCl64 = Debug|NaCl64
add NaCl32

http://codereview.chromium.org/11150028/diff/1/visual_studio/NativeClientVSAd...
File
visual_studio/NativeClientVSAddIn/InstallerResources/examples/hello_nacl/hello_nacl/hello_nacl.c
(right):

http://codereview.chromium.org/11150028/diff/1/visual_studio/NativeClientVSAd...
visual_studio/NativeClientVSAddIn/InstallerResources/examples/hello_nacl/hello_nacl/hello_nacl.c:61:
// - Build STEP6 and verify the file <project
nit: line break here seems a little odd

http://codereview.chromium.org/11150028/diff/1/visual_studio/NativeClientVSAd...
visual_studio/NativeClientVSAddIn/InstallerResources/examples/hello_nacl/hello_nacl/hello_nacl.c:138:
void initInstanceInBrowserWindow() {
capitalize

http://codereview.chromium.org/11150028/diff/1/visual_studio/NativeClientVSAd...
visual_studio/NativeClientVSAddIn/InstallerResources/examples/hello_nacl/hello_nacl/hello_nacl.c:142:
ppb_messaging_interface->PostMessage(myInstance, CStrToVar("Hello, Native
don't line-break in string

http://codereview.chromium.org/11150028/diff/1/visual_studio/NativeClientVSAd...
visual_studio/NativeClientVSAddIn/InstallerResources/examples/hello_nacl/hello_nacl/hello_nacl.c:163:
initInstanceInPCWindow();
capitalize

http://codereview.chromium.org/11150028/diff/1/visual_studio/NativeClientVSAd...
visual_studio/NativeClientVSAddIn/InstallerResources/examples/hello_nacl/hello_nacl/hello_nacl.c:204:

remove extra newlines

http://codereview.chromium.org/11150028/diff/1/visual_studio/NativeClientVSAd...
visual_studio/NativeClientVSAddIn/InstallerResources/examples/hello_nacl/hello_nacl/hello_nacl.c:283:
MessageBox(NULL,
NULL on next line, indent by 4

http://codereview.chromium.org/11150028/diff/1/visual_studio/NativeClientVSAd...
visual_studio/NativeClientVSAddIn/InstallerResources/examples/hello_nacl/hello_nacl/hello_nacl.c:302:
NULL
close paren on this line

http://codereview.chromium.org/11150028/diff/1/visual_studio/NativeClientVSAd...
visual_studio/NativeClientVSAddIn/InstallerResources/examples/hello_nacl/hello_nacl/hello_nacl.c:306:
MessageBox(NULL,
NULL on next line, indent by 4

http://codereview.chromium.org/11150028/diff/1/visual_studio/NativeClientVSAd...
visual_studio/NativeClientVSAddIn/InstallerResources/examples/hello_nacl/hello_nacl/hello_nacl.c:328:
LRESULT CALLBACK WndProc(HWND hWnd, UINT message, WPARAM wParam, LPARAM lParam)
brace on this line

http://codereview.chromium.org/11150028/diff/1/visual_studio/NativeClientVSAd...
visual_studio/NativeClientVSAddIn/InstallerResources/examples/hello_nacl/hello_nacl/hello_nacl.c:356:
HINSTANCE g_hInstance = NULL; HWND g_hWnd = NULL; int initInstanceInPCWindow()
one line per declaration

http://codereview.chromium.org/11150028/diff/1/visual_studio/NativeClientVSAd...
visual_studio/NativeClientVSAddIn/InstallerResources/examples/hello_nacl/hello_nacl/hello_nacl.c:357:
{ WNDCLASSEX winClass; MSG        uMsg;
brace on previous line, declarations have their own line

http://codereview.chromium.org/11150028/diff/1/visual_studio/NativeClientVSAd...
visual_studio/NativeClientVSAddIn/InstallerResources/examples/hello_nacl/hello_nacl/hello_nacl.c:377:
g_hWnd = CreateWindowEx(NULL, _T("MY_WINDOWS_CLASS"),
params start on next line, one per line (or grouped if associated), 4 indent

http://codereview.chromium.org/11150028/diff/1/visual_studio/NativeClientVSAd...
File
visual_studio/NativeClientVSAddIn/InstallerResources/examples/hello_nacl/hello_nacl/hello_nacl.nmf
(right):

http://codereview.chromium.org/11150028/diff/1/visual_studio/NativeClientVSAd...
visual_studio/NativeClientVSAddIn/InstallerResources/examples/hello_nacl/hello_nacl/hello_nacl.nmf:7:
"url": "NaCl64/newlib/Debug/hello_nacl_64.nexe"
hello_nacl_32.nexe

http://codereview.chromium.org/11150028/diff/1/visual_studio/NativeClientVSAd...
File
visual_studio/NativeClientVSAddIn/InstallerResources/examples/hello_nacl/hello_nacl/index.html
(right):

http://codereview.chromium.org/11150028/diff/1/visual_studio/NativeClientVSAd...
visual_studio/NativeClientVSAddIn/InstallerResources/examples/hello_nacl/hello_nacl/index.html:12:
function pageDidLoad() {
indent 2

http://codereview.chromium.org/11150028/diff/1/visual_studio/NativeClientVSAd...
visual_studio/NativeClientVSAddIn/InstallerResources/examples/hello_nacl/hello_nacl/index.html:16:
var forceRedraw = function (element) {
kill this function

http://codereview.chromium.org/11150028/diff/1/visual_studio/NativeClientVSAd...
visual_studio/NativeClientVSAddIn/InstallerResources/examples/hello_nacl/hello_nacl/index.html:32:
function appendStatus(opt_message) {
replace this with implementation from SDK examples: see
load_progress/example.js: appendToEventLog for an example. We don't need to add
new elements for each status item, just replace the innerHTML.

http://codereview.chromium.org/11150028/diff/1/visual_studio/NativeClientVSAd...
visual_studio/NativeClientVSAddIn/InstallerResources/examples/hello_nacl/hello_nacl/index.html:42:

remove all extra newlines

http://codereview.chromium.org/11150028/diff/1/visual_studio/NativeClientVSAd...
File
visual_studio/NativeClientVSAddIn/InstallerResources/examples/hello_nacl_cpp/hello_nacl_cpp/hello_nacl_cpp.cpp
(right):

http://codereview.chromium.org/11150028/diff/1/visual_studio/NativeClientVSAd...
visual_studio/NativeClientVSAddIn/InstallerResources/examples/hello_nacl_cpp/hello_nacl_cpp/hello_nacl_cpp.cpp:100:
int initInstanceInPCWindow();
capitalize

http://codereview.chromium.org/11150028/diff/1/visual_studio/NativeClientVSAd...
visual_studio/NativeClientVSAddIn/InstallerResources/examples/hello_nacl_cpp/hello_nacl_cpp/hello_nacl_cpp.cpp:101:
void initInstanceInBrowserWindow();
capitalize

http://codereview.chromium.org/11150028/diff/1/visual_studio/NativeClientVSAd...
visual_studio/NativeClientVSAddIn/InstallerResources/examples/hello_nacl_cpp/hello_nacl_cpp/hello_nacl_cpp.cpp:111:
public:
indent 1 space

http://codereview.chromium.org/11150028/diff/1/visual_studio/NativeClientVSAd...
visual_studio/NativeClientVSAddIn/InstallerResources/examples/hello_nacl_cpp/hello_nacl_cpp/hello_nacl_cpp.cpp:112:
pp::CompletionCallbackFactory<NaClProjectInstance> factory_;
data members at end of class

http://codereview.chromium.org/11150028/diff/1/visual_studio/NativeClientVSAd...
visual_studio/NativeClientVSAddIn/InstallerResources/examples/hello_nacl_cpp/hello_nacl_cpp/hello_nacl_cpp.cpp:115:
: pp::Instance(instance),
indent 4

http://codereview.chromium.org/11150028/diff/1/visual_studio/NativeClientVSAd...
visual_studio/NativeClientVSAddIn/InstallerResources/examples/hello_nacl_cpp/hello_nacl_cpp/hello_nacl_cpp.cpp:126:
const char* /*argv*/[]) {
indent to align with ( from previous line

http://codereview.chromium.org/11150028/diff/1/visual_studio/NativeClientVSAd...
visual_studio/NativeClientVSAddIn/InstallerResources/examples/hello_nacl_cpp/hello_nacl_cpp/hello_nacl_cpp.cpp:161:
private:
indent 1 space

http://codereview.chromium.org/11150028/diff/1/visual_studio/NativeClientVSAd...
visual_studio/NativeClientVSAddIn/InstallerResources/examples/hello_nacl_cpp/hello_nacl_cpp/hello_nacl_cpp.cpp:168:
public:
indent 1 space

http://codereview.chromium.org/11150028/diff/1/visual_studio/NativeClientVSAd...
visual_studio/NativeClientVSAddIn/InstallerResources/examples/hello_nacl_cpp/hello_nacl_cpp/hello_nacl_cpp.cpp:178:
Module* CreateModule() {
no indent for namespaces

http://codereview.chromium.org/11150028/diff/1/visual_studio/NativeClientVSAd...
visual_studio/NativeClientVSAddIn/InstallerResources/examples/hello_nacl_cpp/hello_nacl_cpp/hello_nacl_cpp.cpp:187:
#ifdef STEP1
same fixes from the C version of this file...?

http://codereview.chromium.org/11150028/diff/1/visual_studio/NativeClientVSAd...
File
visual_studio/NativeClientVSAddIn/InstallerResources/examples/hello_nacl_cpp/hello_nacl_cpp/hello_nacl_cpp.nmf
(right):

http://codereview.chromium.org/11150028/diff/1/visual_studio/NativeClientVSAd...
visual_studio/NativeClientVSAddIn/InstallerResources/examples/hello_nacl_cpp/hello_nacl_cpp/hello_nacl_cpp.nmf:7:
"url": "NaCl64/newlib/Debug/hello_nacl_cpp_64.nexe"
32

http://codereview.chromium.org/11150028/diff/1/visual_studio/NativeClientVSAd...
File
visual_studio/NativeClientVSAddIn/InstallerResources/examples/hello_nacl_cpp/hello_nacl_cpp/index.html
(right):

http://codereview.chromium.org/11150028/diff/1/visual_studio/NativeClientVSAd...
visual_studio/NativeClientVSAddIn/InstallerResources/examples/hello_nacl_cpp/hello_nacl_cpp/index.html:9:
<meta http-equiv="Pragma" content="no-cache">
same fixes from C version?

[Sam Clegg, 2012-10-15 22:54:52]

http://codereview.chromium.org/11150028/diff/1/visual_studio/NativeClientVSAd...
File
visual_studio/NativeClientVSAddIn/InstallerResources/examples/hello_nacl/hello_nacl.sln
(right):

http://codereview.chromium.org/11150028/diff/1/visual_studio/NativeClientVSAd...
visual_studio/NativeClientVSAddIn/InstallerResources/examples/hello_nacl/hello_nacl.sln:8:
Debug|NaCl64 = Debug|NaCl64
On 2012/10/15 22:15:46, binji wrote:
> add NaCl32

Done.

http://codereview.chromium.org/11150028/diff/1/visual_studio/NativeClientVSAd...
File
visual_studio/NativeClientVSAddIn/InstallerResources/examples/hello_nacl/hello_nacl/hello_nacl.c
(right):

http://codereview.chromium.org/11150028/diff/1/visual_studio/NativeClientVSAd...
visual_studio/NativeClientVSAddIn/InstallerResources/examples/hello_nacl/hello_nacl/hello_nacl.c:61:
// - Build STEP6 and verify the file <project
On 2012/10/15 22:15:46, binji wrote:
> nit: line break here seems a little odd

Done.

http://codereview.chromium.org/11150028/diff/1/visual_studio/NativeClientVSAd...
visual_studio/NativeClientVSAddIn/InstallerResources/examples/hello_nacl/hello_nacl/hello_nacl.c:138:
void initInstanceInBrowserWindow() {
On 2012/10/15 22:15:46, binji wrote:
> capitalize

Done.

http://codereview.chromium.org/11150028/diff/1/visual_studio/NativeClientVSAd...
visual_studio/NativeClientVSAddIn/InstallerResources/examples/hello_nacl/hello_nacl/hello_nacl.c:142:
ppb_messaging_interface->PostMessage(myInstance, CStrToVar("Hello, Native
On 2012/10/15 22:15:46, binji wrote:
> don't line-break in string

Done.

http://codereview.chromium.org/11150028/diff/1/visual_studio/NativeClientVSAd...
visual_studio/NativeClientVSAddIn/InstallerResources/examples/hello_nacl/hello_nacl/hello_nacl.c:163:
initInstanceInPCWindow();
On 2012/10/15 22:15:46, binji wrote:
> capitalize

Done.

http://codereview.chromium.org/11150028/diff/1/visual_studio/NativeClientVSAd...
visual_studio/NativeClientVSAddIn/InstallerResources/examples/hello_nacl/hello_nacl/hello_nacl.c:204:

On 2012/10/15 22:15:46, binji wrote:
> remove extra newlines

Done.

http://codereview.chromium.org/11150028/diff/1/visual_studio/NativeClientVSAd...
visual_studio/NativeClientVSAddIn/InstallerResources/examples/hello_nacl/hello_nacl/hello_nacl.c:283:
MessageBox(NULL,
On 2012/10/15 22:15:46, binji wrote:
> NULL on next line, indent by 4

Done.

http://codereview.chromium.org/11150028/diff/1/visual_studio/NativeClientVSAd...
visual_studio/NativeClientVSAddIn/InstallerResources/examples/hello_nacl/hello_nacl/hello_nacl.c:302:
NULL
On 2012/10/15 22:15:46, binji wrote:
> close paren on this line

Done.

http://codereview.chromium.org/11150028/diff/1/visual_studio/NativeClientVSAd...
visual_studio/NativeClientVSAddIn/InstallerResources/examples/hello_nacl/hello_nacl/hello_nacl.c:306:
MessageBox(NULL,
On 2012/10/15 22:15:46, binji wrote:
> NULL on next line, indent by 4

Done.

http://codereview.chromium.org/11150028/diff/1/visual_studio/NativeClientVSAd...
visual_studio/NativeClientVSAddIn/InstallerResources/examples/hello_nacl/hello_nacl/hello_nacl.c:328:
LRESULT CALLBACK WndProc(HWND hWnd, UINT message, WPARAM wParam, LPARAM lParam)
On 2012/10/15 22:15:46, binji wrote:
> brace on this line

Done.

http://codereview.chromium.org/11150028/diff/1/visual_studio/NativeClientVSAd...
visual_studio/NativeClientVSAddIn/InstallerResources/examples/hello_nacl/hello_nacl/hello_nacl.c:356:
HINSTANCE g_hInstance = NULL; HWND g_hWnd = NULL; int initInstanceInPCWindow()
On 2012/10/15 22:15:46, binji wrote:
> one line per declaration

Done.

http://codereview.chromium.org/11150028/diff/1/visual_studio/NativeClientVSAd...
visual_studio/NativeClientVSAddIn/InstallerResources/examples/hello_nacl/hello_nacl/hello_nacl.c:357:
{ WNDCLASSEX winClass; MSG        uMsg;
On 2012/10/15 22:15:46, binji wrote:
> brace on previous line, declarations have their own line

Done.

http://codereview.chromium.org/11150028/diff/1/visual_studio/NativeClientVSAd...
visual_studio/NativeClientVSAddIn/InstallerResources/examples/hello_nacl/hello_nacl/hello_nacl.c:377:
g_hWnd = CreateWindowEx(NULL, _T("MY_WINDOWS_CLASS"),
On 2012/10/15 22:15:46, binji wrote:
> params start on next line, one per line (or grouped if associated), 4 indent

Done.

http://codereview.chromium.org/11150028/diff/1/visual_studio/NativeClientVSAd...
File
visual_studio/NativeClientVSAddIn/InstallerResources/examples/hello_nacl/hello_nacl/hello_nacl.nmf
(right):

http://codereview.chromium.org/11150028/diff/1/visual_studio/NativeClientVSAd...
visual_studio/NativeClientVSAddIn/InstallerResources/examples/hello_nacl/hello_nacl/hello_nacl.nmf:7:
"url": "NaCl64/newlib/Debug/hello_nacl_64.nexe"
On 2012/10/15 22:15:46, binji wrote:
> hello_nacl_32.nexe

nmf is now auto-generated.

http://codereview.chromium.org/11150028/diff/1/visual_studio/NativeClientVSAd...
File
visual_studio/NativeClientVSAddIn/InstallerResources/examples/hello_nacl/hello_nacl/index.html
(right):

http://codereview.chromium.org/11150028/diff/1/visual_studio/NativeClientVSAd...
visual_studio/NativeClientVSAddIn/InstallerResources/examples/hello_nacl/hello_nacl/index.html:12:
function pageDidLoad() {
On 2012/10/15 22:15:46, binji wrote:
> indent 2

Done.

http://codereview.chromium.org/11150028/diff/1/visual_studio/NativeClientVSAd...
visual_studio/NativeClientVSAddIn/InstallerResources/examples/hello_nacl/hello_nacl/index.html:16:
var forceRedraw = function (element) {
On 2012/10/15 22:15:46, binji wrote:
> kill this function

I'd rather leave that to Mark.  He seems to think its needed :(

http://codereview.chromium.org/11150028/diff/1/visual_studio/NativeClientVSAd...
visual_studio/NativeClientVSAddIn/InstallerResources/examples/hello_nacl/hello_nacl/index.html:16:
var forceRedraw = function (element) {
On 2012/10/15 22:15:46, binji wrote:
> kill this function

Done.

http://codereview.chromium.org/11150028/diff/1/visual_studio/NativeClientVSAd...
visual_studio/NativeClientVSAddIn/InstallerResources/examples/hello_nacl/hello_nacl/index.html:42:

On 2012/10/15 22:15:46, binji wrote:
> remove all extra newlines

Done.

http://codereview.chromium.org/11150028/diff/1/visual_studio/NativeClientVSAd...
File
visual_studio/NativeClientVSAddIn/InstallerResources/examples/hello_nacl_cpp/hello_nacl_cpp/hello_nacl_cpp.cpp
(right):

http://codereview.chromium.org/11150028/diff/1/visual_studio/NativeClientVSAd...
visual_studio/NativeClientVSAddIn/InstallerResources/examples/hello_nacl_cpp/hello_nacl_cpp/hello_nacl_cpp.cpp:100:
int initInstanceInPCWindow();
On 2012/10/15 22:15:46, binji wrote:
> capitalize

Done.

http://codereview.chromium.org/11150028/diff/1/visual_studio/NativeClientVSAd...
visual_studio/NativeClientVSAddIn/InstallerResources/examples/hello_nacl_cpp/hello_nacl_cpp/hello_nacl_cpp.cpp:101:
void initInstanceInBrowserWindow();
On 2012/10/15 22:15:46, binji wrote:
> capitalize

Done.

http://codereview.chromium.org/11150028/diff/1/visual_studio/NativeClientVSAd...
visual_studio/NativeClientVSAddIn/InstallerResources/examples/hello_nacl_cpp/hello_nacl_cpp/hello_nacl_cpp.cpp:101:
void initInstanceInBrowserWindow();
On 2012/10/15 22:15:46, binji wrote:
> capitalize

Done.

http://codereview.chromium.org/11150028/diff/1/visual_studio/NativeClientVSAd...
visual_studio/NativeClientVSAddIn/InstallerResources/examples/hello_nacl_cpp/hello_nacl_cpp/hello_nacl_cpp.cpp:111:
public:
On 2012/10/15 22:15:46, binji wrote:
> indent 1 space

Done.

http://codereview.chromium.org/11150028/diff/1/visual_studio/NativeClientVSAd...
visual_studio/NativeClientVSAddIn/InstallerResources/examples/hello_nacl_cpp/hello_nacl_cpp/hello_nacl_cpp.cpp:112:
pp::CompletionCallbackFactory<NaClProjectInstance> factory_;
On 2012/10/15 22:15:46, binji wrote:
> data members at end of class

Done.

http://codereview.chromium.org/11150028/diff/1/visual_studio/NativeClientVSAd...
visual_studio/NativeClientVSAddIn/InstallerResources/examples/hello_nacl_cpp/hello_nacl_cpp/hello_nacl_cpp.cpp:115:
: pp::Instance(instance),
On 2012/10/15 22:15:46, binji wrote:
> indent 4

Done.

http://codereview.chromium.org/11150028/diff/1/visual_studio/NativeClientVSAd...
visual_studio/NativeClientVSAddIn/InstallerResources/examples/hello_nacl_cpp/hello_nacl_cpp/hello_nacl_cpp.cpp:126:
const char* /*argv*/[]) {
On 2012/10/15 22:15:46, binji wrote:
> indent to align with ( from previous line

Done.

http://codereview.chromium.org/11150028/diff/1/visual_studio/NativeClientVSAd...
visual_studio/NativeClientVSAddIn/InstallerResources/examples/hello_nacl_cpp/hello_nacl_cpp/hello_nacl_cpp.cpp:161:
private:
On 2012/10/15 22:15:46, binji wrote:
> indent 1 space

Done.

http://codereview.chromium.org/11150028/diff/1/visual_studio/NativeClientVSAd...
visual_studio/NativeClientVSAddIn/InstallerResources/examples/hello_nacl_cpp/hello_nacl_cpp/hello_nacl_cpp.cpp:161:
private:
On 2012/10/15 22:15:46, binji wrote:
> indent 1 space

Done.

http://codereview.chromium.org/11150028/diff/1/visual_studio/NativeClientVSAd...
visual_studio/NativeClientVSAddIn/InstallerResources/examples/hello_nacl_cpp/hello_nacl_cpp/hello_nacl_cpp.cpp:161:
private:
On 2012/10/15 22:15:46, binji wrote:
> indent 1 space

Done.

http://codereview.chromium.org/11150028/diff/1/visual_studio/NativeClientVSAd...
visual_studio/NativeClientVSAddIn/InstallerResources/examples/hello_nacl_cpp/hello_nacl_cpp/hello_nacl_cpp.cpp:168:
public:
On 2012/10/15 22:15:46, binji wrote:
> indent 1 space

Done.

http://codereview.chromium.org/11150028/diff/1/visual_studio/NativeClientVSAd...
visual_studio/NativeClientVSAddIn/InstallerResources/examples/hello_nacl_cpp/hello_nacl_cpp/hello_nacl_cpp.cpp:178:
Module* CreateModule() {
On 2012/10/15 22:15:46, binji wrote:
> no indent for namespaces

Done.

http://codereview.chromium.org/11150028/diff/1/visual_studio/NativeClientVSAd...
File
visual_studio/NativeClientVSAddIn/InstallerResources/examples/hello_nacl_cpp/hello_nacl_cpp/hello_nacl_cpp.nmf
(right):

http://codereview.chromium.org/11150028/diff/1/visual_studio/NativeClientVSAd...
visual_studio/NativeClientVSAddIn/InstallerResources/examples/hello_nacl_cpp/hello_nacl_cpp/hello_nacl_cpp.nmf:7:
"url": "NaCl64/newlib/Debug/hello_nacl_cpp_64.nexe"
On 2012/10/15 22:15:46, binji wrote:
> 32

Done.

[binji, 2012-10-15 23:03:07]

lgtm

http://codereview.chromium.org/11150028/diff/9001/visual_studio/NativeClientV...
File
visual_studio/NativeClientVSAddIn/InstallerResources/examples/hello_nacl/hello_nacl/hello_nacl.c
(right):

http://codereview.chromium.org/11150028/diff/9001/visual_studio/NativeClientV...
visual_studio/NativeClientVSAddIn/InstallerResources/examples/hello_nacl/hello_nacl/hello_nacl.c:353:
int InitInstanceInPCWindow()
brace on this line

http://codereview.chromium.org/11150028/diff/9001/visual_studio/NativeClientV...
File
visual_studio/NativeClientVSAddIn/InstallerResources/examples/hello_nacl_cpp/hello_nacl_cpp/hello_nacl_cpp.cc
(right):

http://codereview.chromium.org/11150028/diff/9001/visual_studio/NativeClientV...
visual_studio/NativeClientVSAddIn/InstallerResources/examples/hello_nacl_cpp/hello_nacl_cpp/hello_nacl_cpp.cc:202:
int nCmdShow)
brace this line

http://codereview.chromium.org/11150028/diff/9001/visual_studio/NativeClientV...
visual_studio/NativeClientVSAddIn/InstallerResources/examples/hello_nacl_cpp/hello_nacl_cpp/hello_nacl_cpp.cc:259:
LRESULT CALLBACK WndProc(HWND hWnd, UINT message, WPARAM wParam, LPARAM lParam)
brace this line