Hi
Thanks for the clarification, though still a question: if we stop supporting
literal by the unary operators, then wouldn't this be a regression? As an
example the documentation (http://camel.apache.org/simple) says the
following for "--":
To decrement a number by one. The left hand side must be a function,
otherwise parsed as literal.
Or do I misunderstand the documentation?
Babak
Claus Ibsen-2 wrote
> The unary operators should really only be used on functions.
>
> And when people do logging they may use dashes in their logs, eg as
> this example with -------
> And in these cases we should improve the parser, to be more lenient
> whether its an unary operator in use or not.
>
> So I am improving this to only apply unary operators if the left hand
> side is a function. And that the operator is followed by a whitespace
> or EOL.
>
> Then you can still do inc counters
>
> eg some people needs this:
> <setHeader headerName="counter">
>
> <simple>
> ${counter}++
> </simple>
> </setHeader>
> And for dec (which possible isnt as often used as ++)
> <setHeader headerName="counter">
>
> <simple>
> ${counter}--
> </simple>
> </setHeader>
>
> On Mon, Jul 8, 2013 at 9:56 PM, Babak Vahdat
> <
> babak.vahdat@
> > wrote:
>> Hi
>>
>> To my understanding this is "worked as designed" and we should close
>> CAMEL-6414 as "Not a Problem" because the unary operator String "--" is
>> part
>> of the language itself so that Camel simple parser tries to decrement isA
>> by
>> 1 which apparently fails as the conversion of isA to a java.lang.Number
>> fails (see
>> https://github.com/apache/camel/blob/master/camel-core/src/main/java/org/apache/camel/language/simple/ast/UnaryExpression.java#L114).
>> For example if you would change the ignored test you've added as
>> following
>> then it would run green:
>>
>> …
>> SimpleExpressionParser parser = new SimpleExpressionParser("THE
>> MSG
>> ID ${header.JMSMessageID}3--", true);
>> …
>> assertEquals("THE MSG ID JMSMessageID-1232",
>> exp.evaluate(exchange,
>> String.class));
>>
>> As you see we just decremented 3 by 1 which results to 2. In this case we
>> decremented a literal but as well you can also decrement a function, see
>> SimpleDecHeaderTest for an example of this.
>>
>> I'm sure Claus can correct me if I'm wrong here :-)
>>
>> Babak
>>
>>
>> Christian Mueller wrote
>>> Hadrian, go ahead for doing this release...
>>>
>>> I reported one of the issues and added an unit test (ignored at
>>> present),
>>> but I didn't assigned the ticket to myself. I think we have other
>>> committers who could fix this with less effort than me...
>>>
>>> Best,
>>> Christian
>>> -----------------
>>>
>>> Software Integration Specialist
>>>
>>> Apache Camel committer: https://camel.apache.org/team
>>> V.P. Apache Camel: https://www.apache.org/foundation/
>>> Apache Member: https://www.apache.org/foundation/members.html
>>>
>>> https://www.linkedin.com/pub/christian-mueller/11/551/642
>>>
>>>
>>> On Mon, Jul 8, 2013 at 5:24 PM, Hadrian Zbarcea <
>>
>>> hzbarcea@
>>
>>> > wrote:
>>>
>>>> I could do it too, got back from vacation.
>>>>
>>>> One of the issues is yours Christian, I will look into it after
>>>> finishing
>>>> this build. The other one is just doc, I think.
>>>>
>>>> Running a full test now, all good so far (almost forgot how long it
>>>> takes
>>>> :( ).
>>>>
>>>> Hadrian
>>>>
>>>>
>>>> On 07/08/2013 10:59 AM, Christian Müller wrote:
>>>>
>>>>> I can do it. What's with the two outstanding issues? Could you have a
>>>>> look
>>>>> and fix them or postpone them to 2.11.2?
>>>>>
>>>>> Best,
>>>>> Christian
>>>>>
>>>>> Sent from a mobile device
>>>>> Am 08.07.2013 14:58 schrieb "Claus Ibsen" <
>>
>>> claus.ibsen@
>>
>>> >:
>>>>>
>>>>> Hi
>>>>>>
>>>>>> Christian do you have time to work on the 2.11.1 release? Would be
>>>>>> great to get this out as well.
>>>>>>
>>>>>>
>>>>>> On Mon, Jul 1, 2013 at 12:09 AM, Christian Müller
>>>>>> <
>>
>>> christian.mueller@
>>
>>> > wrote:
>>>>>>
>>>>>>> Thanks Hadrian for the offer. If we are not able to start the
>>>>>>> release
>>>>>>>
>>>>>> this
>>>>>>
>>>>>>> week, I'm happy to take this over to you :-).
>>>>>>>
>>>>>>> I updated the maven release plugin configuration for the 2.11.x
>>>>>>>
>>>>>> maintenance
>>>>>>
>>>>>>> branch to use the Git repo and merged Dan's change to only generate
>>>>>>> the
>>>>>>> HTML manual. I tested it and it works fine.
>>>>>>>
>>>>>>> Claus and me pushed a few issues to Camel 2.11.2 and now, we only
>>>>>>> have
>>>>>>> 9
>>>>>>> outstanding issues scheduled for Camel 2.11.1.
>>>>>>> 2 of these issues are classified as bug and should be fixed.
>>>>>>> 2 other issues addressing missing documentation for new components
>>>>>>> (ical
>>>>>>> and scala-extraz).
>>>>>>>
>>>>>>> All other issues are 'nice to have' in Camel 2.11.1, but not
>>>>>>> 'required',
>>>>>>>
>>>>>> in
>>>>>>
>>>>>>> my opinion.
>>>>>>>
>>>>>>> Best,
>>>>>>> Christian
>>>>>>> -----------------
>>>>>>>
>>>>>>> Software Integration Specialist
>>>>>>>
>>>>>>> Apache Camel committer: https://camel.apache.org/team
>>>>>>> V.P. Apache Camel:
>>>>>>> https://www.apache.org/**foundation/<https://www.apache.org/foundation/>
>>>>>>> Apache Member:
>>>>>>> https://www.apache.org/**foundation/members.html<https://www.apache.org/foundation/members.html>
>>>>>>>
>>>>>>> https://www.linkedin.com/pub/**christian-mueller/11/551/642<https://www.linkedin.com/pub/christian-mueller/11/551/642>
>>>>>>>
>>>>>>>
>>>>>>> On Fri, Jun 28, 2013 at 12:06 AM, Hadrian Zbarcea <
>>
>>> hzbarcea@
>>
>>> >>>> wrote:
>>>>>>>
>>>>>>> Hi Christian,
>>>>>>>>
>>>>>>>> Next week I'll be in vacation, but I can do the 2.11.1 after I come
>>>>>>>>
>>>>>>> back.
>>>>>>
>>>>>>> It should be ok, I assume the 22 issues will not be all resolved
>>>>>>> next
>>>>>>>>
>>>>>>> week.
>>>>>>
>>>>>>>
>>>>>>>> Cheers,
>>>>>>>> Hadrian
>>>>>>>>
>>>>>>>>
>>>>>>>>
>>>>>>>> On 06/27/2013 05:02 PM, Christian Müller wrote:
>>>>>>>>
>>>>>>>> We agreed to not build the PDF manual starting with Camel 2.12.0.
>>>>>>>> We
>>>>>>>>>
>>>>>>>> still
>>>>>>
>>>>>>> have to discuss whether the HTML manual should still created or not.
>>>>>>>>> However, because we only publish the HTML/PDF manual for
>>>>>>>>> major/minor
>>>>>>>>> releases, it's not important for Camel 2.11.1.
>>>>>>>>>
>>>>>>>>> We have 22 unresolved issues assigned to Camel 2.11.1 [1]. Could
>>>>>>>>>
>>>>>>>> everybody
>>>>>>
>>>>>>> please have a look and mention the issues which should be included
>>>>>>> in
>>>>>>>>>
>>>>>>>> this
>>>>>>
>>>>>>> release. I will move all others to 2.11.2, probably at the
>>>>>>> weekend...
>>>>>>>>>
>>>>>>>>> And I could do the release or support the release manager, if
>>>>>>>>> someone
>>>>>>>>>
>>>>>>>> else
>>>>>>
>>>>>>> would like to do it.
>>>>>>>>>
>>>>>>>>> [1]
>>>>>>>>> https://issues.apache.org/****jira/issues/?jql=project%20%**
>>>>>>>>> 3D%20CAMEL%20AND%20resolution%****20%3D%20Unresolved%20AND%**
>>>>>>>>> 20fixVersion%20%3D%20%222.11.****1%22<
>>>>>>>>>
>>>>>>>> https://issues.apache.org/**jira/issues/?jql=project%20%**
>>>>>> 3D%20CAMEL%20AND%20resolution%**20%3D%20Unresolved%20AND%**
>>>>>> 20fixVersion%20%3D%20%222.11.**1%22<https://issues.apache.org/jira/issues/?jql=project%20%3D%20CAMEL%20AND%20resolution%20%3D%20Unresolved%20AND%20fixVersion%20%3D%20%222.11.1%22>
>>>>>>
>>>>>>>
>>>>>>>
>>>>>>>>> Best,
>>>>>>>>> Christian
>>>>>>>>> -----------------
>>>>>>>>>
>>>>>>>>> Software Integration Specialist
>>>>>>>>>
>>>>>>>>> Apache Camel committer: https://camel.apache.org/team
>>>>>>>>> V.P. Apache Camel:
>>>>>>>>> https://www.apache.org/****foundation/<https://www.apache.org/**foundation/>
>>>>>>>>> <
>>>>>>>>>
>>>>>>>> https://www.apache.org/**foundation/<https://www.apache.org/foundation/>
>>>>>> >
>>>>>>
>>>>>>> Apache Member:
>>>>>>> https://www.apache.org/****foundation/members.html<https://www.apache.org/**foundation/members.html>
>>>>>>>>> <
>>>>>>>>>
>>>>>>>> https://www.apache.org/**foundation/members.html<https://www.apache.org/foundation/members.html>
>>>>>> >
>>>>>>
>>>>>>>
>>>>>>>>> https://www.linkedin.com/pub/****christian-mueller/11/551/642<https://www.linkedin.com/pub/**christian-mueller/11/551/642>
>>>>>>>>> <
>>>>>>>>>
>>>>>>>> https://www.linkedin.com/pub/**christian-mueller/11/551/642<https://www.linkedin.com/pub/christian-mueller/11/551/642>
>>>>>> >
>>>>>>
>>>>>>>
>>>>>>>>>
>>>>>>>>> On Thu, Jun 27, 2013 at 3:21 PM, Daniel Kulp <
>>
>>> dkulp@
>>
>>> >
>>>>>>>>> wrote:
>>>>>>>>>
>>>>>>>>>
>>>>>>>>> On Jun 27, 2013, at 6:04 AM, Claus Ibsen <
>>
>>> claus.ibsen@
>>
>>> >
>>>>>>>>>>
>>>>>>>>> wrote:
>>>>>>
>>>>>>>
>>>>>>>>>> Hi
>>>>>>>>>>
>>>>>>>>>>>
>>>>>>>>>>> We recently released the 2.10.5 release after our git migration.
>>>>>>>>>>> It would be great if we could start working on a 2.11.1 release
>>>>>>>>>>> as
>>>>>>>>>>>
>>>>>>>>>> well.
>>>>>>
>>>>>>>
>>>>>>>>>>>
>>>>>>>>>> I certainly support the idea of a 2.11.1 release, but we *DO*
>>>>>>>>>> need
>>>>>>>>>> to
>>>>>>>>>> figure out what we want to do with the PDF manual first. We can
>>>>>>>>>> no
>>>>>>>>>> longer
>>>>>>>>>> produce a usable manual.
>>>>>>>>>>
>>>>>>>>>> Dan
>>>>>>>>>>
>>>>>>>>>>
>>>>>>>>>>
>>>>>>>>>>
>>>>>>>>>>>
>>>>>>>>>>> --
>>>>>>>>>>> Claus Ibsen
>>>>>>>>>>> -----------------
>>>>>>>>>>> www.camelone.org: The open source integration conference.
>>>>>>>>>>>
>>>>>>>>>>> Red Hat, Inc.
>>>>>>>>>>> FuseSource is now part of Red Hat
>>>>>>>>>>> Email:
>>
>>> cibsen@
>>
>>>>>>>>>>> Web: http://fusesource.com
>>>>>>>>>>> Twitter: davsclaus
>>>>>>>>>>> Blog: http://davsclaus.com
>>>>>>>>>>> Author of Camel in Action: http://www.manning.com/ibsen
>>>>>>>>>>>
>>>>>>>>>>>
>>>>>>>>>> --
>>>>>>>>>> Daniel Kulp
>>>>>>>>>>
>>
>>> dkulp@
>>
>>> - http://dankulp.com/blog
>>>>>>>>>> Talend Community Coder - http://coders.talend.com
>>>>>>>>>>
>>>>>>>>>>
>>>>>>>>>>
>>>>>>>>>>
>>>>>>>>>
>>>>>>
>>>>>>
>>>>>> --
>>>>>> Claus Ibsen
>>>>>> -----------------
>>>>>> Red Hat, Inc.
>>>>>> Email:
>>
>>> cibsen@
>>
>>>>>> Twitter: davsclaus
>>>>>> Blog: http://davsclaus.com
>>>>>> Author of Camel in Action: http://www.manning.com/ibsen
>>>>>>
>>>>>>
>>>>>
>>
>>
>>
>>
>>
>> --
>> View this message in context:
>> http://camel.465427.n5.nabble.com/DISCUSS-Apache-Camel-2-11-1-release-tp5734891p5735309.html
>> Sent from the Camel Development mailing list archive at Nabble.com.
>
>
>
> --
> Claus Ibsen
> -----------------
> Red Hat, Inc.
> Email:
> cibsen@
> Twitter: davsclaus
> Blog: http://davsclaus.com
> Author of Camel in Action: http://www.manning.com/ibsen
--
View this message in context:
http://camel.465427.n5.nabble.com/DISCUSS-Apache-Camel-2-11-1-release-tp5734891p5735344.html
Sent from the Camel Development mailing list archive at Nabble.com.
