Looking at the recent separation of API and IMPL for the Pluto driver

[Charles Severance <cs...@umich.edu>]

Why are these still in the directory that *should* consist only of  
interfaces?

charles-severances-computer:~/dev/pluto-1.1/pluto-trunk/pluto-portal- 
driver csev$ grep -ir implements * | grep -v svn

src/main/java/org/apache/pluto/driver/core/PortletWindowIDImpl.java:
      public class PortletWindowIDImpl implements PortletWindowID,  
Serializable {

src/main/java/org/apache/pluto/driver/core/PortletWindowImpl.java:
     public class PortletWindowImpl implements PortletWindow {

src/main/java/org/apache/pluto/driver/core/ResourceURLProviderImpl.java:
     public class ResourceURLProviderImpl implements  
ResourceURLProvider {

src/main/java/org/apache/pluto/driver/PortalStartupListener.java:
    public class PortalStartupListener implements  
ServletContextListener {

src/main/java/org/apache/pluto/driver/url/PortalURL.java:
     public class PortalURL implements Cloneable {

charles-severances-computer:~/dev/pluto-1.1/pluto-trunk/pluto-portal- 
driver csev$

Re: Looking at the recent separation of API and IMPL for the Pluto driver

["David H. DeWolf" <dd...@apache.org>]

In general it's either because:

1) They require the extraction of an additional interface and/or factory 
before they are abstracted.

OR

2) I just flat out missed.

The commit that I've done was simply the first step of the process. 
More to come this week on this front. . .


David

Charles Severance wrote:
> Why are these still in the directory that *should* consist only of 
> interfaces?
> 
> charles-severances-computer:~/dev/pluto-1.1/pluto-trunk/pluto-portal-driver 
> csev$ grep -ir implements * | grep -v svn
> 
> src/main/java/org/apache/pluto/driver/core/PortletWindowIDImpl.java:
>      public class PortletWindowIDImpl implements PortletWindowID, 
> Serializable {
> 
> src/main/java/org/apache/pluto/driver/core/PortletWindowImpl.java:
>     public class PortletWindowImpl implements PortletWindow {
> 
> src/main/java/org/apache/pluto/driver/core/ResourceURLProviderImpl.java:
>     public class ResourceURLProviderImpl implements ResourceURLProvider {
> 
> src/main/java/org/apache/pluto/driver/PortalStartupListener.java:
>    public class PortalStartupListener implements ServletContextListener {
> 
> src/main/java/org/apache/pluto/driver/url/PortalURL.java:
>     public class PortalURL implements Cloneable {
> 
> charles-severances-computer:~/dev/pluto-1.1/pluto-trunk/pluto-portal-driver 
> csev$
> 
> 
>