Skip to content

Commit

Permalink
rebase-i: style fix
Browse files Browse the repository at this point in the history
Case arms should align with "case" and "esac".

Do not cat a file into a pipeline; just make the downstream command
read from the file.

Having a while statement as a downstream of a pipe is fine, but
the loop should begin on its own line.

Signed-off-by: Junio C Hamano <gitster@pobox.com>
  • Loading branch information
Junio C Hamano committed Jul 6, 2010
1 parent 869d588 commit 41f556b
Showing 1 changed file with 6 additions and 5 deletions.
11 changes: 6 additions & 5 deletions git-rebase--interactive.sh
Original file line number Diff line number Diff line change
Expand Up @@ -263,10 +263,10 @@ pick_one_preserving_merges () {
then
if test "$fast_forward" = t
then
cat "$DOTEST"/current-commit | while read current_commit
while read current_commit
do
git rev-parse HEAD > "$REWRITTEN"/$current_commit
done
done <"$DOTEST"/current-commit
rm "$DOTEST"/current-commit ||
die "Cannot write current commit's replacement sha1"
fi
Expand Down Expand Up @@ -440,9 +440,9 @@ record_in_rewritten() {
echo "$oldsha1" >> "$REWRITTEN_PENDING"

case "$(peek_next_command)" in
squash|s|fixup|f)
squash|s|fixup|f)
;;
*)
*)
flush_rewritten_pending
;;
esac
Expand Down Expand Up @@ -890,7 +890,8 @@ first and then run 'git rebase --continue' again."
git rev-list $MERGES_OPTION --pretty=oneline --abbrev-commit \
--abbrev=7 --reverse --left-right --topo-order \
$REVISIONS | \
sed -n "s/^>//p" | while read shortsha1 rest
sed -n "s/^>//p" |
while read shortsha1 rest
do
if test t != "$PRESERVE_MERGES"
then
Expand Down

0 comments on commit 41f556b

Please sign in to comment.