Jean-Marc Lasgouttes wrote:
"Abdelrazak" == Abdelrazak Younes <[EMAIL PROTECTED]> writes:

Abdelrazak> What do you think of the attached. I even simplified some
Abdelrazak> expand functions...

Simplified? The patch seems rather mechanical to me... What changes do you
have in mind?

I just mean that I simplified the interface of these expand functions (no Buffer passing) that's all.


I think I'd still prefer a single add that does the checks, since the
only reason for not passing view was lazyness on my part. Why do you
want to maintain this separation? I think this hardcoding based on
knowledge on the LFUN inner working is bad, unless we can _prove_ that
there is a speed advantage.

I am not sure I understand what you mean here. Do you want a single function that does the check in any case? If yes, I am OK with that too and I don't think that there will be a noticeable performance penalty.

Abdel.

Reply via email to