[ 
https://issues.apache.org/jira/browse/GSHELL-92?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12573190#action_12573190
 ] 

Jason Dillon commented on GSHELL-92:
------------------------------------

What about the annotations that are used to configure XStream on the other 
objects?  Having them present requires XStream anyways na?  I agree that the 
the loading bits should be moved to a {{LayoutLoader}} (or something) class 
though.

> Move the XStream specific bits of the Layout in a helper class
> --------------------------------------------------------------
>
>                 Key: GSHELL-92
>                 URL: https://issues.apache.org/jira/browse/GSHELL-92
>             Project: GShell
>          Issue Type: Improvement
>      Security Level: public(Regular issues) 
>    Affects Versions: 1.0-alpha-1
>            Reporter: Guillaume Nodet
>            Assignee: Jason Dillon
>             Fix For: 1.0-alpha-2
>
>
> The Layout itself does not have any dependency on XStream, so when you're not 
> using an XStream based layout, it's bit weird to have to include this 
> dependecy.  It should be moved to a helper class so that the default layout 
> loader can use it.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

Reply via email to