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

Issue 29727558: Issue 6489 - fix initialization moment of the default scheduler of DefaultPlatformBuilder (Closed)

Created:
March 19, 2018, 10:17 a.m. by sergei
Modified:
March 19, 2018, 2:36 p.m.
Reviewers:
René Jeschke, hub
CC:
anton
Base URL:
https://github.com/adblockplus/libadblockplus@cd1cd70ea3122fb4b8a96c40d96ce9b1ae5a3ae9
Visibility:
Public.

Patch Set 1 #

Total comments: 3

Patch Set 2 : fix typo #

Unified diffs Side-by-side diffs Delta from patch set Stats (+73 lines, -34 lines) Patch
M include/AdblockPlus/AsyncExecutor.h View 1 1 chunk +51 lines, -1 line 0 comments Download
M include/AdblockPlus/Platform.h View 3 chunks +9 lines, -0 lines 0 comments Download
M src/Platform.cpp View 5 chunks +13 lines, -33 lines 0 comments Download

Messages

Total messages: 4
sergei
https://codereview.adblockplus.org/29727558/diff/29727559/include/AdblockPlus/Platform.h File include/AdblockPlus/Platform.h (right): https://codereview.adblockplus.org/29727558/diff/29727559/include/AdblockPlus/Platform.h#newcode143 include/AdblockPlus/Platform.h:143: * Private If you find it confusing I can ...
March 19, 2018, 10:22 a.m. (2018-03-19 10:22:49 UTC) #1
hub
beside the small nit LGTM https://codereview.adblockplus.org/29727558/diff/29727559/include/AdblockPlus/AsyncExecutor.h File include/AdblockPlus/AsyncExecutor.h (right): https://codereview.adblockplus.org/29727558/diff/29727559/include/AdblockPlus/AsyncExecutor.h#newcode100 include/AdblockPlus/AsyncExecutor.h:100: * Destroyes internally held ...
March 19, 2018, 11:48 a.m. (2018-03-19 11:48:43 UTC) #2
sergei
https://codereview.adblockplus.org/29727558/diff/29727559/include/AdblockPlus/AsyncExecutor.h File include/AdblockPlus/AsyncExecutor.h (right): https://codereview.adblockplus.org/29727558/diff/29727559/include/AdblockPlus/AsyncExecutor.h#newcode100 include/AdblockPlus/AsyncExecutor.h:100: * Destroyes internally held `AsyncExecutor`, any subsequent calls of ...
March 19, 2018, 12:34 p.m. (2018-03-19 12:34:27 UTC) #3
hub
March 19, 2018, 1:10 p.m. (2018-03-19 13:10:38 UTC) #4
LGTM

Powered by Google App Engine
This is Rietveld