Milan Vancura wrote:

> > Milan, did you read the earlier post? It seems to deal with that.
> 
> Thank you for the reference, Ian. I made O(n) variant, two patches follows as
> attachemnts. Bram, are they OK for you, please?
> 
> The first one fasten the join operation, the second one removes do_do_join()
> function as it is no longer needed. They are separate to allow to accept just
> the first one if API change is not allowed etc.

Thanks for the patch.

I haven't looked into the details, but you should at least use spaces
like in the rest of the code.

Did you do "make test"?  Test 29 should be extended with joining
multiple lines.  Currently it does one line at a time only.

-- 
<Beeth> Girls are like internet domain names,
        the ones I like are already taken.
<honx>  Well, you can stil get one from a strange country :-P

 /// Bram Moolenaar -- b...@moolenaar.net -- http://www.Moolenaar.net   \\\
///        sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ \\\
\\\        download, build and distribute -- http://www.A-A-P.org        ///
 \\\            help me help AIDS victims -- http://ICCF-Holland.org    ///

--~--~---------~--~----~------------~-------~--~----~
You received this message from the "vim_dev" maillist.
For more information, visit http://www.vim.org/maillist.php
-~----------~----~----~----~------~----~------~--~---

Raspunde prin e-mail lui