Re: svn commit: r1061310 - in /ofbiz/trunk/framework: webapp/src/org/ofbiz/webapp/control/ControlServlet.java widget/src/org/ofbiz/widget/screen/ScreenRenderer.java

Previous Topic Next Topic
 
classic Classic list List threaded Threaded
2 messages Options
Reply | Threaded
Open this post in threaded view
|

Re: svn commit: r1061310 - in /ofbiz/trunk/framework: webapp/src/org/ofbiz/webapp/control/ControlServlet.java widget/src/org/ofbiz/widget/screen/ScreenRenderer.java

Erwan de FERRIERES
Le 20/01/2011 15:23, [hidden email] a écrit :
> -
> +

You've just added spaces on this one... I doubt this is what you wanted
to do !

Cheers,
--
Erwan de FERRIERES
www.nereide.biz
Reply | Threaded
Open this post in threaded view
|

Re: svn commit: r1061310 - in /ofbiz/trunk/framework: webapp/src/org/ofbiz/webapp/control/ControlServlet.java widget/src/org/ofbiz/widget/screen/ScreenRenderer.java

Jacques Le Roux
Administrator
Thanks Erwan,

Actually it has already been commited by Hans at r890175 when he merged the Birt branch. But as, I did not know at this time and the
issue was not closed, I thought it still needed to be committed. Then it was not a sole patch but 2 diffs. So I concatenated them
and applied the resulting patch with Tortoise (because Subclipse refused and I though it was because of my concatenation, so I tried
a merge with Tortoise and successed). So I thought it was good. But then when I compiled it refused again. I found the 2 duplicated
snippets and removed them and thought another time that it was because of the merge with Tortoise...

I'm a bit dissapointed because I posted a message 2 days ago about that, asking about this issue...

Jacques

From: "Erwan de FERRIERES" <[hidden email]>
Subject: Re: svn commit: r1061310 - in /ofbiz/trunk/framework: webapp/src/org/ofbiz/webapp/control/ControlServlet.java
widget/src/org/ofbiz/widget/screen/ScreenRenderer.java

> Le 20/01/2011 15:23, [hidden email] a écrit :
>> -
>> +
>
> You've just added spaces on this one... I doubt this is what you wanted to do !
>
> Cheers,
> --
> Erwan de FERRIERES
> www.nereide.biz
>