pijul/pijul - Change D2YXPJIY5E4TEPUQIAJGTNVE3JJISUVTXYSNSOALUQE546WV44EQC

Fix the hunk_roundtrip test, simplify code, improve test coverage, fix some edge cases.

Created by FHRXP5Jnb2MWLDrPrnLnkN2ryWcGCo6CRr1dXR9FW2YA on December 9, 2021

D2YXPJIY5E4TEPUQIAJGTNVE3JJISUVTXYSNSOALUQE546WV44EQC

Dependencies

In discussions

#585 Fix the hunk_roundtrip test

Change contents

Replacement in libpijul/src/change/printable.rs at line 415 [3.36296]

B:BD[3.47117] → [4.142:293]

                if !content_edges.is_empty() {
                    writeln!(w, "{}", PrintableAtom::Edges(content_edges.to_vec()))?;
                }

[3.47117]

[3.47199]

                writeln!(w, "{}", PrintableAtom::Edges(content_edges.to_vec()))?;

Replacement in libpijul/src/change/printable.rs at line 434 [3.36296]

B:BD[3.47786] → [4.294:445]

                if !content_edges.is_empty() {
                    writeln!(w, "{}", PrintableAtom::Edges(content_edges.to_vec()))?;
                }

[3.47786]

[3.47868]

                writeln!(w, "{}", PrintableAtom::Edges(content_edges.to_vec()))?;

Replacement in libpijul/src/change/printable.rs at line 456 [3.36296]

B:BD[5.73] → [5.73:116]

                    if e[0].flag.deleted {

[5.73]

[5.116]

                    if e.is_empty() || e[0].flag.deleted {

Insertion in libpijul/src/change/printable.rs at line 698 [3.36296]

[3.55511]

            AddRoot {
                start: f(g),
            },
            DelRoot {
                name: f(g), inode: f(g)
            },

Replacement in libpijul/src/change/printable.rs at line 708 [3.36296]

B:BD[3.55556] → [3.55556:55759]

    // Shrinking frequently blows stack. Investigate how to fix it.
    // You can disable shrinking by commenting out this function.
    // This may be best solved by switching to proptest crate
    /*

[3.55556]

[3.55759]

    // Shrinking frequently blows stack.
    // See https://github.com/BurntSushi/quickcheck/pull/294.
    // We can't really solve this problem without introducing a ton of boilerplate.

Insertion in libpijul/src/change/printable.rs at line 712 [3.36296]
[3.55816]
[3.55816]
```
        use std::iter::empty;
```

Replacement in libpijul/src/change/printable.rs at line 714 [3.36296]

B:BD[3.55845] → [3.55845:56198]

B:BD[3.56198] → [3.56198:56199]

B:BD[3.56199] → [3.56199:56816]

            FileMoveV { path, name, perms, pos, up_context, down_context, del } =>
                Box::new((path, name, perms, pos, up_context, down_context, del)
                .shrink().map(|(path, name, perms, pos, up_context, down_context, del)|
                fix_encoding(FileMoveV { path, name, perms, pos, up_context, down_context, del }))),
            FileMoveE { path, pos, add, del } =>
                Box::new((path, pos, add, del)
                .shrink().map(|(path, pos, add, del)|
                fix_encoding(FileMoveE { path, pos, add, del }))),
            FileAddition { name, parent, perms, encoding, up_context, start, end, contents } =>
                Box::new((name, parent, perms, encoding, up_context, start, end, contents)
                .shrink().map(|(name, parent, perms, encoding, up_context, start, end, contents)|
                fix_encoding(FileAddition { name, parent, perms, encoding, up_context, start, end, contents }))),

[3.55845]

[3.56816]

            // Example of shrinking for future reference:

Replacement in libpijul/src/change/printable.rs at line 716 [3.36296]

B:BD[3.56817] → [3.56817:59826]

            FileDel { path, pos, encoding, del_edges, content_edges, contents } =>
                Box::new((path, pos, encoding, del_edges, content_edges, contents)
                .shrink().map(|(path, pos, encoding, del_edges, content_edges, contents)|
                fix_encoding(FileDel { path, pos, encoding, del_edges, content_edges, contents }))),
            FileUndel { path, pos, encoding, undel_edges, content_edges, contents } =>
                Box::new((path, pos, encoding, undel_edges, content_edges, contents)
                .shrink().map(|(path, pos, encoding, undel_edges, content_edges, contents)|
                fix_encoding(FileUndel { path, pos, encoding, undel_edges, content_edges, contents }))),
            Edit { path, line, pos, encoding, change, contents } =>
                Box::new((path, line, pos, encoding, change, contents)
                .shrink().map(|(path, line, pos, encoding, change, contents)|
                fix_encoding(Edit { path, line, pos, encoding, change, contents }))),
            Replace { path, line, pos, encoding, change, replacement, change_contents, replacement_contents } =>
                Box::new((path, line, pos, encoding, change, replacement, change_contents, replacement_contents)
                .shrink().map(|(path, line, pos, encoding, change, replacement, change_contents, replacement_contents)|
                fix_encoding(Replace { path, line, pos, encoding, change, replacement, change_contents, replacement_contents }))),
            SolveNameConflict { path, pos, names, edges } =>
                Box::new((path, pos, names, edges)
                .shrink().map(|(path, pos, names, edges)|
                fix_encoding(SolveNameConflict { path, pos, names, edges }))),
            UnsolveNameConflict { path, pos, names, edges } =>
                Box::new((path, pos, names, edges)
                .shrink().map(|(path, pos, names, edges)|
                fix_encoding(UnsolveNameConflict { path, pos, names, edges }))),
            SolveOrderConflict { path, line, pos, encoding, change, contents } =>
                Box::new((path, line, pos, encoding, change, contents)
                .shrink().map(|(path, line, pos, encoding, change, contents)|
                fix_encoding(SolveOrderConflict { path, line, pos, encoding, change, contents }))),
            UnsolveOrderConflict { path, line, pos, encoding, change, contents } =>
                Box::new((path, line, pos, encoding, change, contents)
                .shrink().map(|(path, line, pos, encoding, change, contents)|
                fix_encoding(UnsolveOrderConflict { path, line, pos, encoding, change, contents }))),
            ResurrectZombies { path, line, pos, encoding, change, contents } =>
                Box::new((path, line, pos, encoding, change, contents)
                .shrink().map(|(path, line, pos, encoding, change, contents)|
                fix_encoding(ResurrectZombies { path, line, pos, encoding, change, contents }))),

[3.56817]

[3.59826]

            // FileMoveV { path, name, perms, pos, up_context, down_context, del } =>
            //     Box::new((path, name, perms, pos, up_context, down_context, del)
            //     .shrink().map(|(path, name, perms, pos, up_context, down_context, del)|
            //     fix_encoding(FileMoveV { path, name, perms, pos, up_context, down_context, del }))),
            _ => Box::new(empty()),

Deletion in libpijul/src/change/printable.rs at line 723 [3.36296]
B:BD[3.59842] → [3.59842:59849]
```
    */
```

Replacement in libpijul/src/change/parse.rs at line 68 [3.64713]

B:BD[3.66660] → [2.0:115]

    let (i, content_edges) = if let Ok(x) = parse_edges(i) {
        x
    } else {
        (i, Vec::new())
    };

[3.66660]

[3.66706]

    let (i, content_edges) = map(opt(parse_edges), |o| o.unwrap_or(Vec::new()))(i)?;

Replacement in libpijul/src/change/parse.rs at line 92 [3.64713]

B:BD[3.67368] → [2.116:231]

    let (i, content_edges) = if let Ok(x) = parse_edges(i) {
        x
    } else {
        (i, Vec::new())
    };

[3.67368]

[3.67414]

    let (i, content_edges) = map(opt(parse_edges), |o| o.unwrap_or(Vec::new()))(i)?;

Replacement in libpijul/src/change/parse.rs at line 119 [3.64713]

B:BD[3.68289] → [2.232:389]

    let (i, contents) = if let PrintablePerms::IsDir = perms {
        (i, Vec::new())
    } else {
        parse_contents('+', encoding.clone(), i)?
    };

[3.68289]

[3.68356]

    let (i, contents) = parse_contents('+', encoding.clone(), i)?;

Replacement in libpijul/src/change/parse.rs at line 144 [3.64713]

B:BD[3.69024] → [5.354:436]

B:BD[5.436] → [2.390:513]

    let (i, contents) = if let Ok(s) = parse_contents('+', encoding.clone(), i) {
        if s.1.is_empty() {
            parse_contents('-', encoding.clone(), i)?
        } else {
            s
        }

[3.69024]

[5.446]

    let (i, contents_plus) = parse_contents('+', encoding.clone(), i)?;
    let (i, contents_minus) = parse_contents('-', encoding.clone(), i)?;
    let contents = if contents_plus.is_empty() {
        contents_minus

Replacement in libpijul/src/change/parse.rs at line 149 [3.64713]
B:BD[5.459] → [5.459:509]
```
        parse_contents('-', encoding.clone(), i)?
```
[5.459]
[5.509]
```
        contents_plus
```
Deletion in libpijul/src/change/parse.rs at line 358 [3.64713]
B:BD[3.74568] → [6.293:336]
```
    let has_encoding = encoding.is_some();
```

Replacement in libpijul/src/change/parse.rs at line 359 [3.64713]

B:BD[3.74618] → [2.514:721]

∅:D[6.382] → [7.1000:1033]

∅:D[2.721] → [7.1000:1033]

B:BD[5.546] → [7.1000:1033]

        let not_empty = if backslash.is_some() && vec[vec.len() - 1] == b'\n' {
            vec.pop().is_some()
        } else {
            !vec.is_empty()
        };
        if has_encoding || not_empty {
            return Ok((i, vec));

[3.74618]

[5.578]

        if backslash.is_some() && vec[vec.len() - 1] == b'\n' {
            vec.pop();

Insertion in libpijul/src/change/parse.rs at line 362 [3.64713]

[5.588]

[3.74871]

        Ok((i, vec))
    } else {
        Err(nom::Err::Error(nom::error::Error::new(
            i,
            nom::error::ErrorKind::Verify,
        )))

Deletion in libpijul/src/change/parse.rs at line 369 [3.64713]

B:BD[3.74877] → [5.589:695]

    Err(nom::Err::Error(nom::error::Error::new(
        i,
        nom::error::ErrorKind::Verify,
    )))