Skip to main content

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index] [List Home]
Re: [tracecompass-dev] Let's Lambda!

I have no issues with that, except for trivial lambdas.
My definition of a trivial lambda?

()->{true;}
or worst case:
(Object o)->{o.toString()}

Objections?

On 16-04-12 03:40 PM, Patrick Tasse wrote:
> I'm not against lambdas although it's a pity that due to the change in
> indentation it will affect a lot of the git blame annotations.
>
> But I would very much prefer that we avoid one-liner lambdas (e.g. put
> the lambda body in a block on its own line) so that we can put
> breakpoints and/or use Ctrl+R (Run to Line) when debugging.
>
> Patrick
>
> On Mon, Apr 11, 2016 at 1:06 PM, Matthew Khouzam
> <matthew.khouzam@xxxxxxxxxxxx <mailto:matthew.khouzam@xxxxxxxxxxxx>>
> wrote:
>
>     For those who want to go deeper.
>
>     http://www.infoq.com/articles/Java-8-Lambdas-A-Peek-Under-the-Hood
>
>     On 16-04-11 01:05 PM, Matthew Khouzam wrote:
>     > So I was thinking about how great our code looks, and we need to
>     keep up
>     > our efforts to keep it as clean as possible. I am going to suggest
>     > something, feel free to shoot it down.
>     >
>     > If you are writing an anon class or a listener, consider using a
>     lambda
>     > expression instead. It will not make an object and therefore should
>     > reduce the quantity of objects allocated.
>     >
>     > The lambda will also encourage statelessness and therefore
>     potentially
>     > reduce leaks.
>     >
>     > Finally it will make our code more compact. I am not saying it's
>     more
>     > readable, but we can get more code on our screen at any moment,
>     and that
>     > is a good thing.
>     >
>     > Thoughts?
>     > _______________________________________________
>     > tracecompass-dev mailing list
>     > tracecompass-dev@xxxxxxxxxxx <mailto:tracecompass-dev@xxxxxxxxxxx>
>     > To change your delivery options, retrieve your password, or
>     unsubscribe from this list, visit
>     > https://dev.eclipse.org/mailman/listinfo/tracecompass-dev
>
>     _______________________________________________
>     tracecompass-dev mailing list
>     tracecompass-dev@xxxxxxxxxxx <mailto:tracecompass-dev@xxxxxxxxxxx>
>     To change your delivery options, retrieve your password, or
>     unsubscribe from this list, visit
>     https://dev.eclipse.org/mailman/listinfo/tracecompass-dev
>
>
>
>
> _______________________________________________
> tracecompass-dev mailing list
> tracecompass-dev@xxxxxxxxxxx
> To change your delivery options, retrieve your password, or unsubscribe from this list, visit
> https://dev.eclipse.org/mailman/listinfo/tracecompass-dev



Back to the top