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

Issue 7729045: Fix debugger build errors/warnings (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
11 years, 9 months ago by robertphillips
Modified:
11 years, 9 months ago
Reviewers:
DerekS
CC:
skia-review_googlegroups.com
Base URL:
http://skia.googlecode.com/svn/trunk/
Visibility:
Public.

Description

Don't know why but the debugger stopped compiling in Windows when I updated.

Patch Set 1 #

Patch Set 2 : Readd functionality from r8025 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+22 lines, -16 lines) Patch
debugger/QT/SkDebuggerGUI.cpp View 1 3 chunks +6 lines, -4 lines 0 comments Download
debugger/SkDebugger.h View 1 1 chunk +2 lines, -1 line 0 comments Download
debugger/SkDebugger.cpp View 1 4 chunks +14 lines, -11 lines 0 comments Download

Messages

Total messages: 5
robertphillips
11 years, 9 months ago (2013-03-12 14:52:32 UTC) #1
DerekS
lgtm
11 years, 9 months ago (2013-03-12 15:01:05 UTC) #2
robertphillips
On further investigation, r8095 (https://codereview.chromium.org/12449006) overwrote some of the changes made in r8025 (Normalize the ...
11 years, 9 months ago (2013-03-12 15:25:28 UTC) #3
DerekS
lgtm
11 years, 9 months ago (2013-03-12 15:32:01 UTC) #4
robertphillips
11 years, 9 months ago (2013-03-12 15:33:56 UTC) #5
Message was sent while issue was closed.
committed as r8105
Sign in to reply to this message.

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