Merge pull request #17 from andyyu2004/builder-finish-fix

Fix trailing_newline issue in Builder::build

src/rope/rope_builder.rs

@@ -107,12 +107,12 @@ impl RopeBuilder {
     #[inline]
     pub fn build(mut self) -> Rope {
         if self.buffer_len_left > 0 {
-            self.rope_has_trailing_newline =
-                self.buffer.has_trailing_newline();
-
             self.buffer.left_summary =
                 ChunkSummary::from(self.buffer_left_chunk());
 
+            self.rope_has_trailing_newline =
+                self.buffer.has_trailing_newline();
+
             self.tree_builder.append(self.buffer);
         }
 

src/tree/tree.rs

@@ -1548,7 +1548,6 @@ mod tests {
     use core::ops::{Add, AddAssign, Sub, SubAssign};
 
     use super::*;
-    use crate::tree::Summarize;
 
     #[derive(Copy, Clone, Default, Debug, Eq, PartialEq)]
     pub struct Count {

tests/rope_builder.rs

@@ -3,6 +3,14 @@ mod common;
 use common::LARGE;
 use crop::{Rope, RopeBuilder};
 
+#[test]
+fn builder_line_len() {
+    let mut builder = RopeBuilder::new();
+    builder.append("\n");
+    let rope = builder.build();
+    assert_eq!(rope.line_len(), Rope::from("\n").line_len());
+}
+
 #[test]
 fn builder_empty() {
     let r = RopeBuilder::new().build();