ok/jj
1
0
Fork 0
forked from mirrors/jj
Code Pull requests Activity

git-push: remove redundant hash map lookup from print-changes loop

Browse source
This commit is contained in:
Yuya Nishihara 2024-09-14 11:45:44 +09:00
parent 2f78c25d35
commit a48ecf3e62
1 changed files with 10 additions and 22 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

32
cli/src/commands/git/push.rs
View file

@ -12,7 +12,6 @@
// See the License for the specific language governing permissions and
// limitations under the License.
use std::collections::HashMap;
use std::collections::HashSet;
use std::fmt;
use std::io;
@ -384,27 +383,16 @@ fn print_commits_ready_to_push(
repo: &dyn Repo,
bookmark_updates: &[(String, BookmarkPushUpdate)],
) -> io::Result<()> {
let mut bookmark_push_direction = HashMap::new();
for (bookmark_name, update) in bookmark_updates {
let BookmarkPushUpdate {
old_target: Some(old_target),
new_target: Some(new_target),
} = update
else {
continue;
};
let to_direction = |old_target: &CommitId, new_target: &CommitId| {
assert_ne!(old_target, new_target);
bookmark_push_direction.insert(
bookmark_name.to_string(),
if repo.index().is_ancestor(old_target, new_target) {
BookmarkMoveDirection::Forward
} else if repo.index().is_ancestor(new_target, old_target) {
BookmarkMoveDirection::Backward
} else {
BookmarkMoveDirection::Sideways
},
);
}
if repo.index().is_ancestor(old_target, new_target) {
BookmarkMoveDirection::Forward
} else if repo.index().is_ancestor(new_target, old_target) {
BookmarkMoveDirection::Backward
} else {
BookmarkMoveDirection::Sideways
}
};
for (bookmark_name, update) in bookmark_updates {
match (&update.old_target, &update.new_target) {
@ -417,7 +405,7 @@ fn print_commits_ready_to_push(
// among many was moved sideways (say). TODO: People on Discord
// suggest "Move bookmark ... forward by n commits",
// possibly "Move bookmark ... sideways (X forward, Y back)".
let msg = match bookmark_push_direction.get(bookmark_name).unwrap() {
let msg = match to_direction(old_target, new_target) {
BookmarkMoveDirection::Forward => {
format!("Move forward bookmark {bookmark_name} from {old} to {new}")
}