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

Issue 11612008: Bunch of tiny fixes (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
5 years, 11 months ago by Oleksandr
Modified:
5 years, 11 months ago
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): ...
5 years, 11 months ago (2013-09-05 22:47:40 UTC) #1
Wladimir Palant
LGTM
5 years, 11 months ago (2013-09-11 13:08:45 UTC) #2
Felix Dahlke
5 years, 11 months ago (2013-09-11 14:50:43 UTC) #3
LGTM
Sign in to reply to this message.

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