Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code

Issue 11612008: Bunch of tiny fixes (Closed)

Created:
Sept. 5, 2013, 10:38 p.m. by Oleksandr
Modified:
Sept. 16, 2013, 7:05 a.m.
Visibility:
Public.

Description

Bunch of tiny fixes

Patch Set 1 #

Total comments: 3
Unified diffs Side-by-side diffs Delta from patch set Stats (+11 lines, -13 lines) Patch
M src/engine/Main.cpp View 1 chunk +2 lines, -2 lines 1 comment Download
M src/plugin/PluginSettings.h View 2 chunks +1 line, -2 lines 0 comments Download
M src/plugin/PluginSettings.cpp View 1 chunk +4 lines, -4 lines 1 comment Download
M src/plugin/PluginTabBase.cpp View 1 chunk +4 lines, -4 lines 1 comment Download
M src/shared/Communication.h View 1 chunk +0 lines, -1 line 0 comments Download

Messages

Total messages: 3
Oleksandr
http://codereview.adblockplus.org/11612008/diff/1/src/engine/Main.cpp File src/engine/Main.cpp (right): http://codereview.adblockplus.org/11612008/diff/1/src/engine/Main.cpp#newcode56 src/engine/Main.cpp:56: MessageBoxW(NULL, upToDateText.c_str(), upToDateTitle.c_str(), MB_OK); http://codereview.adblockplus.org/11276031/diff/3001/src/engine/main.cpp#newcode53 http://codereview.adblockplus.org/11612008/diff/1/src/plugin/PluginSettings.cpp File src/plugin/PluginSettings.cpp (right): ...
Sept. 5, 2013, 10:47 p.m. (2013-09-05 22:47:40 UTC) #1
Wladimir Palant
LGTM
Sept. 11, 2013, 1:08 p.m. (2013-09-11 13:08:45 UTC) #2
Felix Dahlke
Sept. 11, 2013, 2:50 p.m. (2013-09-11 14:50:43 UTC) #3
LGTM

Powered by Google App Engine
This is Rietveld