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

Issue 4814063: Remove unnecessary Visit function overloading. (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
14 years, 8 months ago by Zhenyao Mo
Modified:
14 years, 8 months ago
Reviewers:
nicolas
CC:
angleproject-review_googlegroups.com, kbr1, Alok Priyadarshi
Base URL:
http://angleproject.googlecode.com/svn/trunk/
Visibility:
Public.

Description

Remove unnecessary Visit function overloading. BUG=none TEST=build ok, run as before Committed: http://code.google.com/p/angleproject/source/detail?r=715

Patch Set 1 #

Patch Set 2 : '' #

Unified diffs Side-by-side diffs Delta from patch set Stats (+1 line, -112 lines) Patch
M src/common/version.h View 1 1 chunk +1 line, -1 line 0 comments Download
M src/compiler/DetectRecursion.h View 1 1 chunk +0 lines, -7 lines 0 comments Download
M src/compiler/DetectRecursion.cpp View 1 2 chunks +0 lines, -33 lines 0 comments Download
M src/compiler/MapLongVariableNames.h View 1 1 chunk +0 lines, -6 lines 0 comments Download
M src/compiler/MapLongVariableNames.cpp View 1 2 chunks +0 lines, -29 lines 0 comments Download
M src/compiler/ValidateLimitations.h View 1 1 chunk +0 lines, -4 lines 0 comments Download
M src/compiler/ValidateLimitations.cpp View 1 5 chunks +0 lines, -32 lines 0 comments Download

Messages

Total messages: 3
Zhenyao Mo
Please review.
14 years, 8 months ago (2011-08-01 21:43:10 UTC) #1
Zhenyao Mo
On 2011/08/01 21:43:10, Zhenyao Mo wrote: > Please review. Nicolas: ping.
14 years, 8 months ago (2011-08-02 21:55:46 UTC) #2
nicolas
14 years, 8 months ago (2011-08-03 12:03:45 UTC) #3
Looking good.
Sign in to reply to this message.

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