marijan/pijul - Change NG3Z3DOKDZQDQ7UNQEOMXWX2NJKKLM7DKRWW3KIUD7QEECTNQIZQC

roundtrip text encoding when recording

Missing some cases and should potentially avoid the roundtrip but this is much easier given the current record logic

Created by rohan on March 20, 2021

NG3Z3DOKDZQDQ7UNQEOMXWX2NJKKLM7DKRWW3KIUD7QEECTNQIZQC

Dependencies

In channels

main

Change contents

Insertion in libpijul/src/text_encoding.rs at line 23 [3.19]

[3.601]


    pub(crate) fn encode(&self, text: &str) -> Vec<u8> {
        let (encoded, ..) = self.0.encode(text);
        encoded.into_owned().to_vec()
    }

Insertion in libpijul/src/change/text_changes.rs at line 847 [4.37976]
[4.67725]
[4.67725]
```
                    encoding,
```

Replacement in libpijul/src/change/text_changes.rs at line 867 [4.37976]

B:BD[4.68539] → [4.68539:68624]

                                text_changes::parse_line_add(h, contents, contents_)

[4.68539]

[4.68624]

                                text_changes::parse_line_add(h, contents, contents_, encoding)

Replacement in libpijul/src/change/text_changes.rs at line 972 [4.37976]

B:BD[4.73304] → [5.9965:10026]

                Some(Hunk::Edit { ref mut change, .. }) => {

[4.73304]

[4.73367]

                Some(Hunk::Edit {
                    ref mut change,
                    encoding,
                    ..
                }) => {

Replacement in libpijul/src/change/text_changes.rs at line 983 [4.37976]

B:BD[4.73707] → [4.73707:73786]

                            text_changes::parse_line_add(h, change, contents_)

[4.73707]

[4.73786]

                            text_changes::parse_line_add(h, change, contents_, encoding)

Insertion in libpijul/src/change/text_changes.rs at line 1013 [4.37976]
[4.75183]
[4.75183]
```
                    encoding,
```

Replacement in libpijul/src/change/text_changes.rs at line 1021 [4.37976]

B:BD[4.75523] → [4.75523:75600]

                            text_changes::parse_line_add(h, repl, contents_)

[4.75523]

[4.75600]

                            text_changes::parse_line_add(h, repl, contents_, encoding)

Replacement in libpijul/src/change/text_changes.rs at line 1063 [4.37976]

B:BD[4.77707] → [4.77707:77786]

                            text_changes::parse_line_add(h, change, contents_)

[4.77707]

[4.77786]

                            // TODO encoding
                            text_changes::parse_line_add(h, change, contents_, &None)

Replacement in libpijul/src/change/text_changes.rs at line 1251 [4.37976]

B:BD[4.84069] → [4.84069:84191]

pub fn parse_line_add(h: &str, change: &mut NewVertex<Option<Hash>>, contents_: &mut Vec<u8>) {
    let h = h.as_bytes();

[4.84069]

[4.84191]

pub fn parse_line_add(
    h: &str,
    change: &mut NewVertex<Option<Hash>>,
    contents_: &mut Vec<u8>,
    encoding: &Option<Encoding>,
) {
    let h = match encoding {
        Some(encoding) => encoding.encode(h),
        None => String::from(h).into_bytes(),
    };

Deletion in libpijul/src/change/text_changes.rs at line 1317 [4.37976]
B:BD[6.143228] → [2.1:1]