diff options
| author | Nitzan Gur-Furman <nitzan@google.com> | 2023-12-06 14:47:27 +0100 |
|---|---|---|
| committer | Nitzan Gur-Furman <nitzan@google.com> | 2024-01-09 09:14:06 +0100 |
| commit | 8116f66b9c8e032ef0a2c14e5b242e710ae0d323 (patch) | |
| tree | 22659c209fdf4ba5f1d8a4896e91bd59b2eea26c /org.eclipse.jgit/src/org/eclipse/jgit/util/LfsFactory.java | |
| parent | 4d2bdba32687b87a18b0b3a8439a573305ace3ea (diff) | |
| download | jgit-8116f66b9c8e032ef0a2c14e5b242e710ae0d323.tar.gz jgit-8116f66b9c8e032ef0a2c14e5b242e710ae0d323.zip | |
FooterLines: handle extraction from messages without headers
Prior to this change, long subjects of messages with no headers were
treated as headers, and therefore were skipped. In a message of the
form `<long subject>\n\n<footers>`, the footers would then get parsed
as a message, meaning no footers were returned.
After this change, the first lines are skipped only if they match any
of the known headers. The first line ofter the optional headers is then
assumed to be the subject line.
`FooterLineTest` had a few test cases for extracting footers from
messages with no headers. However, there were all with short messages,
so the "skip this line" logic in `RawParseUtils` was never triggered.
Added test case to catch this issue.
Change-Id: I971a1dddf1a9aea094360c3c8fc3b9a8b011bbf9
Issue: Google b/287891316
Diffstat (limited to 'org.eclipse.jgit/src/org/eclipse/jgit/util/LfsFactory.java')
0 files changed, 0 insertions, 0 deletions