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

Issue 4899447913840640: Issue 1106 - Add currentVersion Pref and compareVersions API (Closed)

Created:
Aug. 1, 2014, 12:51 p.m. by Oleksandr
Modified:
Aug. 1, 2014, 1:19 p.m.
Visibility:
Public.

Description

Issue 1106 - Add currentVersion Pref and compareVersions API

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+17 lines, -1 line) Patch
M include/AdblockPlus/FilterEngine.h View 1 chunk +1 line, -0 lines 0 comments Download
M lib/api.js View 1 chunk +5 lines, -0 lines 0 comments Download
M lib/prefs.js View 1 chunk +2 lines, -1 line 0 comments Download
M src/FilterEngine.cpp View 1 chunk +9 lines, -0 lines 0 comments Download

Messages

Total messages: 4
Oleksandr
Aug. 1, 2014, 12:52 p.m. (2014-08-01 12:52:38 UTC) #1
Felix Dahlke
LGTM
Aug. 1, 2014, 12:55 p.m. (2014-08-01 12:55:31 UTC) #2
Wladimir Palant
LGTM
Aug. 1, 2014, 1:04 p.m. (2014-08-01 13:04:07 UTC) #3
Felix Dahlke
Aug. 1, 2014, 1:09 p.m. (2014-08-01 13:09:20 UTC) #4
Note that I would vote for a different commit message here - it's required for
1106, but it's not fixing that issue.

I'd go with this as a commit message:

"
Add currentVersion Pref and compareVersions API

http://codereview.adblockplus.org/4899447913840640
"

Alternatively, feel free to create an issue for this that blocks 1106 - that'd
be the clean way :P

Powered by Google App Engine
This is Rietveld