Merge: Ropefix
authorJean Privat <jean@pryen.org>
Fri, 30 Jan 2015 02:01:18 +0000 (09:01 +0700)
committerJean Privat <jean@pryen.org>
Fri, 30 Jan 2015 02:01:18 +0000 (09:01 +0700)
commit929eeb8226712ede1d199ee9fff684c6ca191804
treed870b846cc84eec99d6a2f9f104732a552f29b38
parent1772b949316cb3878b6717bf3a7ed2f419ba1c0f
parent97f3820a4dae82aa4a6ebb298654002eb2793486
Merge: Ropefix

A few modifications to ropes, I discovered that the `+` routine used to create too many Concat nodes passed a certain point, which inceased the RAM use of Strings unnecessarily (might help with #1106's example program, on my machine the memory went from 200M+ used to 20M).

While I was at it, I removed a few old-style-inits that were no longer necessary.

Some others however I could not remove, maybe there is a way I can do so, but with the spec of the new constructors I'm not entirely sure how to do so without retaining unnneeded information or complexifying instanciation, that'll have to wait.

Pull-Request: #1127
Reviewed-by: Jean Privat <jean@pryen.org>
Reviewed-by: Alexandre Terrasa <alexandre@moz-code.org>
Reviewed-by: Alexis Laferrière <alexis.laf@xymus.net>