[aerogear-dev] Aerogear Android Packages

[Bruno Oliveira]

- Bruno

On Nov 20, 2012, at 1:58 PM, Summers Pittman <supittma at redhat.com> wrote:

>> Working to make id configurable I created two classes Scan (to search for a annotation) and Property (to encapsulate reflection property calls) where I should put these classes? I created some exceptions. We should >have a package for exceptions?
> 
> If Property and Scan and their exceptions are available on an api or meant to be used by the user directly, I would say put them in org.aerogear.android.reflection.  If not, org.aerogear.android.impl.reflection
+1
> 
>> Also I was in doubt whether the classes HeaderAndBody and HttpException should not be moved to the package impl
> 
> I would suggest org.aerogear.android.net  (and if you move Scan et all to .reflection then .impl.core and become .impl.net)

-1 maybe org.aerogear.android.http ?
> 
> I don't think we should have an exception package.  1) There isn't one in the JDK and 2) it breaks the organizational connections between classes and the types of exceptions they can throw
why not? Just because JDK doesn't have it? I can't see any problems on having org.jboss.aerogear.exception package.
> 
> ----- Original Message -----
> From: "Daniel Passos" <daniel at passos.me>
> To: "AeroGear Developer Mailing" <aerogear-dev at lists.jboss.org>
> Sent: Tuesday, November 20, 2012 10:49:19 AM
> Subject: [aerogear-dev] Aerogear Android Packages
> 
> 
> 
> Hi Guys, 
> 
> 
> I'm having some doubts about where to put some classes. 
> 
> 
> Working to make id configurable I created two classes Scan (to search for a annotation) and Property (to encapsulate reflection property calls) where I should put these classes? I created some exceptions. We should have a package for exceptions? 
> 
> 
> Also I was in doubt whether the classes HeaderAndBody and HttpException should not be moved to the package impl 
> 
> 
> wdyt? 
> 
> 
> Att, 
> Daniel Passos 
> _______________________________________________
> aerogear-dev mailing list
> aerogear-dev at lists.jboss.org
> https://lists.jboss.org/mailman/listinfo/aerogear-dev
> _______________________________________________
> aerogear-dev mailing list
> aerogear-dev at lists.jboss.org
> https://lists.jboss.org/mailman/listinfo/aerogear-dev