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

Issue 3146042: Changing mapBinder variable to protected for reuse in subclasses (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
15 years, 3 months ago by gagan.goku
Modified:
15 years, 3 months ago
Reviewers:
dev, plindner1
Base URL:
http://svn.apache.org/repos/asf/shindig/trunk/
Visibility:
Public.

Description

Another small nit missing in previous submit: http://codereview.appspot.com/3175041/

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+1 line, -1 line) Patch
M java/gadgets/src/main/java/org/apache/shindig/gadgets/rewrite/RewriteModule.java View 1 chunk +1 line, -1 line 0 comments Download

Messages

Total messages: 3
anupama.dutta
LGTM. It will be good to mention the small nit in the CL title and ...
15 years, 3 months ago (2010-11-18 06:38:32 UTC) #1
plindner1
lgtm
15 years, 3 months ago (2010-11-18 17:04:29 UTC) #2
gagan.goku
15 years, 3 months ago (2010-11-19 02:43:28 UTC) #3
On 2010/11/18 17:04:29, plindner1 wrote:
> lgtm

Build looks good. Committed as r1036714.
Thanks for the reviews guys.
Sign in to reply to this message.

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