diff options
author | zeripath <art27@cantab.net> | 2021-02-27 18:46:14 +0000 |
---|---|---|
committer | GitHub <noreply@github.com> | 2021-02-27 19:46:14 +0100 |
commit | 3d8b5ad5f37f495e50d83bbad214f8061f8a5ac4 (patch) | |
tree | c4fb9d7b195f9a7af5d2e3fc275fa74accf0d01c /modules/process/manager_test.go | |
parent | 904a26c57c474e0ed7b43dc37269f69b49240301 (diff) | |
download | gitea-3d8b5ad5f37f495e50d83bbad214f8061f8a5ac4.tar.gz gitea-3d8b5ad5f37f495e50d83bbad214f8061f8a5ac4.zip |
Fix a couple of CommentAsPatch issues. (#14804)
* CutDiffAroundLine makes the incorrect assumption that `---` and `+++` always represent part of the header of a diff.
This PR adds a flag to its parsing to prevent this problem and adds a streaming parsing technique to CutDiffAroundLine using an io.pipe instead of just sending data to an unbounded buffer.
Fix #14711
Signed-off-by: Andrew Thornton <art27@cantab.net>
* Handle unquoted comment patch files
When making comment patches unfortunately the patch does not always quote the filename
This makes the diff --git header ambiguous again.
This PR finally adds handling for ambiguity in to parse patch
Fix #14812
Signed-off-by: Andrew Thornton <art27@cantab.net>
* Add in testing for no error
There is no way currently for CutDiffAroundLine in this test to cause an
error however, it should still be tested.
Signed-off-by: Andrew Thornton <art27@cantab.net>
Diffstat (limited to 'modules/process/manager_test.go')
0 files changed, 0 insertions, 0 deletions