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

Issue 29391770: Noissue - Fix sign comparison warnings in tests. (Closed)

Created:
March 22, 2017, 3:32 p.m. by hub
Modified:
March 22, 2017, 4:09 p.m.
Reviewers:
sergei
Base URL:
https://hg.adblockplus.org/libadblockplus/
Visibility:
Public.

Description

Noissue - Fix sign comparison warnings in tests.

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+8 lines, -8 lines) Patch
M test/Notification.cpp View 1 chunk +2 lines, -2 lines 0 comments Download
M test/ReferrerMapping.cpp View 3 chunks +4 lines, -4 lines 0 comments Download
M test/WebRequest.cpp View 2 chunks +2 lines, -2 lines 0 comments Download

Messages

Total messages: 4
hub
March 22, 2017, 3:32 p.m. (2017-03-22 15:32:18 UTC) #1
sergei
LGTM
March 22, 2017, 3:33 p.m. (2017-03-22 15:33:11 UTC) #2
sergei
don't forget about review URL
March 22, 2017, 3:33 p.m. (2017-03-22 15:33:27 UTC) #3
hub
March 22, 2017, 3:34 p.m. (2017-03-22 15:34:36 UTC) #4
Warnings appeared in a recent commit. Let's fix them.

Powered by Google App Engine
This is Rietveld