andybalholm/pijul-export - Change 5ZSJEED77SE3LHU6S6RMBNRZPDDR3TFNXOM5SI23WIHKS6LHEHAAC

main: rework loop over changes

The iteration with an indexing variable appeared to be mostly to be able to retrieve the state of the previous change.

Change to a consuming iterator and keep track of the previous change state in a mutable variable.

Created by laumann on October 11, 2023

5ZSJEED77SE3LHU6S6RMBNRZPDDR3TFNXOM5SI23WIHKS6LHEHAAC

Dependencies

In discussions

#5 Fix clippy warnings

Change contents

Replacement in src/main.rs at line 47 [6.67]

∅:D[4.5695] → [5.98:130]

B:BD[7.1721] → [5.98:130]

B:BD[5.130] → [4.5696:5746]

B:BD[4.5746] → [8.246:303]

B:BD[8.303] → [2.2081:2150]

∅:D[2.2150] → [8.357:556]

B:BD[8.357] → [8.357:556]

    for i in 0..changes.len() {
        sandbox.add_change(&changes[i]).unwrap();
        if !stream.already_exported(&changes[i].state) {
            let files = sandbox.get_files(changes[i].hash).unwrap();
            let parent = if i == 0 {
                None
            } else {
                Some(changes[i - 1].state)
            };
            stream.write_commit(&changes[i], parent, &files);

[4.5695]

[3.900]

    let mut parent = None;
    for ch in changes {
        sandbox.add_change(&ch).unwrap();
        if !stream.already_exported(&ch.state) {
            let files = sandbox.get_files(ch.hash).unwrap();
            stream.write_commit(&ch, parent, &files);

Replacement in src/main.rs at line 59 [6.67]

B:BD[3.1113] → [3.1113:1156]

                    count, changes[i].hash

[3.1113]

[3.1156]

                    count, ch.hash

Insertion in src/main.rs at line 63 [6.67]
[8.566]
[8.566]
```
	parent = Some(ch.state)
```