The problem here is that we need to include other expressions (than a sourceReference `String`) as well, such as a MethodReference. We only have them as template available, so I wanted to generalize here and take the same approach.  You can see this in my last commit.


On 13 apr. 2014, at 12:25, Gunnar Morling <notifications@github.com> wrote:

> In processor/src/main/java/org/mapstruct/ap/conversion/SimpleConversion.java:
> 
> >       */
> > -    protected abstract String getToConversionString(String sourceReference, ConversionContext conversionContext);
> > +    protected abstract String[] getToExpressions( ConversionContext conversionContext );
> I've not yet understood why return two parts here now; i.e. what's the reationale for having separate "open" and "close" expressions? Could you maybe give an example?
> 
> Btw. we should return a more descriptive object instead of an array, i.e. a class which contains the two expressions.
> 
> —
> Reply to this email directly or view it on GitHub.
>