+1

BTW I don't think if an IDE refactoring review is very helpful... jira
could do the job.
my 2cents

Vincent

2009/4/23  <[email protected]>:
> Reviewers: shindig.remailer_gmail.com,
>
> Message:
> SocialSpiException should be deprecated in favor of ProtocolException.
> It only stayed around after the handler move from social-api to common
> to make migration easier.
>
>
>
> Please review this at http://codereview.appspot.com/49062
>
> Affected files:
>  M
> java/social-api/src/main/java/org/apache/shindig/social/opensocial/service/ActivityHandler.java
>  M
> java/social-api/src/main/java/org/apache/shindig/social/opensocial/service/AppDataHandler.java
>  M
> java/social-api/src/main/java/org/apache/shindig/social/opensocial/service/MessageHandler.java
>  M
> java/social-api/src/main/java/org/apache/shindig/social/opensocial/service/PersonHandler.java
>  M
> java/social-api/src/main/java/org/apache/shindig/social/opensocial/spi/ActivityService.java
>  M
> java/social-api/src/main/java/org/apache/shindig/social/opensocial/spi/AppDataService.java
>  M
> java/social-api/src/main/java/org/apache/shindig/social/opensocial/spi/MessageService.java
>  M
> java/social-api/src/main/java/org/apache/shindig/social/opensocial/spi/PersonService.java
>  M
> java/social-api/src/main/java/org/apache/shindig/social/opensocial/spi/SocialSpiException.java
>  M
> java/social-api/src/main/java/org/apache/shindig/social/sample/service/SampleContainerHandler.java
>  M
> java/social-api/src/main/java/org/apache/shindig/social/sample/spi/JsonDbOpensocialService.java
>  M
> java/social-api/src/test/java/org/apache/shindig/social/opensocial/service/ActivityHandlerTest.java
>  M
> java/social-api/src/test/java/org/apache/shindig/social/opensocial/service/AppDataHandlerTest.java
>  M
> java/social-api/src/test/java/org/apache/shindig/social/opensocial/service/MessageHandlerTest.java
>  M
> java/social-api/src/test/java/org/apache/shindig/social/sample/spi/JsonDbOpensocialServiceTest.java
>
>
>

Reply via email to