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

Issue 29339280: No issue - update google test framework (Closed)

Created:
April 1, 2016, 2:32 p.m. by sergei
Modified:
April 1, 2016, 2:55 p.m.
Reviewers:
René Jeschke
CC:
Felix Dahlke, Oleksandr, Eric
Visibility:
Public.

Description

#project: libadblockplus Because of compilation errors using newer compiler from android ndk r10c and r10d we had to update google testing framework. Code review: https://codereview.adblockplus.org/29339280/ # Compilation has been tested with android-ndk-r9d, android-ndk-r10d as well as with running tests compiled by gcc 5.2.1 and MSVS2012.

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+13 lines, -13 lines) Patch
M dependencies View 1 chunk +1 line, -1 line 0 comments Download
M third_party/googletest.gyp View 2 chunks +12 lines, -12 lines 0 comments Download

Messages

Total messages: 2
sergei
April 1, 2016, 2:36 p.m. (2016-04-01 14:36:11 UTC) #1
René Jeschke
April 1, 2016, 2:41 p.m. (2016-04-01 14:41:35 UTC) #2
LGTM

Assuming it works

Powered by Google App Engine
This is Rietveld