magit/magit — pull request #1001 by npostavs (2013-10-17 01:36:43+00:00)
Title: `magit-goto-previous-sibling-section` gets "stuck" between sections

`magit-goto-previous-sibling-section` falls back to `magit-goto-parent-section` if there are no previous siblings. When point is in between sections or on the last line of the buffer, `magit-goto-previous-sibling-section` doesn't move at all because that line belongs to the top status section so there is no parent or siblings.

I fixed this by making it fall back to `magit-goto-previous-section` instead. This is symmetric with `magit-goto-next-sibling-section`.

---

Actually, I think I might prefer if it jumped to the section header instead, but I can't think of a clean to implement that.

What I mean is, given a `magit-status` buffer that looks like this (`!` represents point):
```
Untracked files:
not-tracked
!
```

Without this patch, `M-p` would do nothing, with patch it goes to `not-tracked` line, best might be to go to `Untracked files:` line.