← Back to team overview

zorba-coders team mailing list archive

Re: [Merge] lp:~zorba-coders/zorba/bug1082740_fn_subsequence into lp:zorba

 

Review: Needs Fixing

Looks good, but I think that it would be nice to factor out the (virtual) calls to getDoubleValue().
Not sure if that's measurable, but I don't think that it hurts to give the compiler a hint :)
-- 
https://code.launchpad.net/~zorba-coders/zorba/bug1082740_fn_subsequence/+merge/157755
Your team Zorba Coders is subscribed to branch lp:zorba.


References