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

Issue 2522041: Fixes for subclassing related issues

Can't Edit
Can't Publish+Mail
Start Review
Created:
15 years, 3 months ago by janluehe
Modified:
15 years, 3 months ago
Reviewers:
henry.saputra, chirag, dev-remailer
Base URL:
http://svn.apache.org/repos/asf/shindig/trunk/
Visibility:
Public.

Description

This patch addresses the following subclassing related issues: - Changes the visibility of GadgetRenderingServlet instance fields from private to protected - Moves the readObject method, which sets "initialized" to false during deserialization, and its writeObject companion method from a custom subclass to InjectServlet, which is where "initialized" is defined

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+35 lines, -2 lines) Patch
java/common/src/main/java/org/apache/shindig/common/servlet/InjectedServlet.java View 2 chunks +33 lines, -0 lines 0 comments Download
java/gadgets/src/main/java/org/apache/shindig/gadgets/servlet/GadgetRenderingServlet.java View 1 chunk +2 lines, -2 lines 0 comments Download

Messages

Total messages: 5
janluehe
15 years, 3 months ago (2010-10-14 21:33:44 UTC) #1
chirag
LGTM!
15 years, 3 months ago (2010-10-14 23:27:40 UTC) #2
henry.saputra
LGTM too!
15 years, 3 months ago (2010-10-14 23:44:07 UTC) #3
henry.saputra
LGTM too!
15 years, 3 months ago (2010-10-14 23:44:12 UTC) #4
chirag
15 years, 3 months ago (2010-10-15 18:24:32 UTC) #5
Committed as of r1023057.
Sign in to reply to this message.

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