Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

FileStats method String() triggers "strings: negative Repeat count" #177

Closed
pawelWritesCode opened this issue Sep 30, 2020 · 3 comments
Closed

FileStats method String() triggers "strings: negative Repeat count" #177

pawelWritesCode opened this issue Sep 30, 2020 · 3 comments
Labels
bug Something isn't working good first issue Good for newcomers help wanted Extra attention is needed

Comments

@pawelWritesCode
Copy link

Hello, I'm trying to obtain list of files that were touched(changed, added,removed,modified..) between two commits.
Here is my code

mainBranchCommit, err := repo.CommitObject(baseCommitHash)

if err != nil {
	return modifiedFilesSlice, err
}

featureBranchCommit, err := repo.CommitObject(featureCommitHash)

if err != nil {
	return modifiedFilesSlice, err
}

patch, err := mainBranchCommit.Patch(featureBranchCommit)

if err != nil {
	return modifiedFilesSlice, err
}

fileStats := patch.Stats()
scanner := bufio.NewScanner(strings.NewReader(fileStats.String()))

Line fileStats.String() triggers panic: strings: negative Repeat count.

Is it my approach to problem or bug with library ?
My git: git version 2.20.1 (Apple Git-117)
Go: go version go1.15.1 darwin/amd64
Also I'm adding output from git client so it may help someone debug problem

$ git diff --stat 467d14c1 2f9462ae
 appp-configs/appp/pawel/app-appp-tech/appp.yml                                     |  2 +-
 backend/app/DoctrineMigrations/Version20200930084310.php                           | 39 ---------------------------------------
 backend/app/DoctrineMigrations/Version20200930090608.php                           | 37 -------------------------------------
 backend/app/DoctrineMigrations/Version20200930110322.php                           | 39 ---------------------------------------
 backend/src/ApppBundle/Controller/SrcAll/Get.php                                   |  2 +-
 backend/src/ApppBundle/Controller/SrcAll/GetList.php                               |  2 +-
 backend/src/ApppBundle/Controller/SrcAll/Post.php                                  |  2 +-
 backend/src/ApppBundle/Controller/SrcAll/Put.php                                   | 57 +++++++++++++++++++++++++++++++++++++++++++++++++++++++++
 backend/src/ApppBundle/Controller/TC/Get.php                                       |  2 +-
 backend/src/ApppBundle/Controller/TC/GetList.php                                   |  2 +-
 backend/src/ApppBundle/Controller/TC/Post.php                                      |  2 +-
 backend/src/ApppBundle/Controller/TC/Put.php                                       | 57 +++++++++++++++++++++++++++++++++++++++++++++++++++++++++
 backend/src/ApppBundle/Entity/GlobalFileList.php                                   |  4 ++--
 backend/src/ApppBundle/Entity/SrcAll.php                                           | 57 +++++++++++++++++++++++++++++++++++++++++++--------------
:...skipping...
 appp-configs/appp/pawel/app-appp-tech/appp.yml                                     |  2 +-
 backend/app/DoctrineMigrations/Version20200930084310.php                           | 39 ---------------------------------------
 backend/app/DoctrineMigrations/Version20200930090608.php                           | 37 -------------------------------------
 backend/app/DoctrineMigrations/Version20200930110322.php                           | 39 ---------------------------------------
 backend/src/ApppBundle/Controller/SrcAll/Get.php                                   |  2 +-
 backend/src/ApppBundle/Controller/SrcAll/GetList.php                               |  2 +-
 backend/src/ApppBundle/Controller/SrcAll/Post.php                                  |  2 +-
 backend/src/ApppBundle/Controller/SrcAll/Put.php                                   | 57 +++++++++++++++++++++++++++++++++++++++++++++++++++++++++
 backend/src/ApppBundle/Controller/TC/Get.php                                       |  2 +-
 backend/src/ApppBundle/Controller/TC/GetList.php                                   |  2 +-
 backend/src/ApppBundle/Controller/TC/Post.php                                      |  2 +-
 backend/src/ApppBundle/Controller/TC/Put.php                                       | 57 +++++++++++++++++++++++++++++++++++++++++++++++++++++++++
 backend/src/ApppBundle/Entity/GlobalFileList.php                                   |  4 ++--
 backend/src/ApppBundle/Entity/SrcAll.php                                           | 57 +++++++++++++++++++++++++++++++++++++++++++--------------
 backend/src/ApppBundle/Entity/TC.php                                               | 57 +++++++++++++++++++++++++++++++++++++++++++--------------
:...skipping...
 appp-configs/appp/pawel/app-appp-tech/appp.yml                                     |  2 +-
 backend/app/DoctrineMigrations/Version20200930084310.php                           | 39 ---------------------------------------
 backend/app/DoctrineMigrations/Version20200930090608.php                           | 37 -------------------------------------
 backend/app/DoctrineMigrations/Version20200930110322.php                           | 39 ---------------------------------------
 backend/src/ApppBundle/Controller/SrcAll/Get.php                                   |  2 +-
 backend/src/ApppBundle/Controller/SrcAll/GetList.php                               |  2 +-
 backend/src/ApppBundle/Controller/SrcAll/Post.php                                  |  2 +-
 backend/src/ApppBundle/Controller/SrcAll/Put.php                                   | 57 +++++++++++++++++++++++++++++++++++++++++++++++++++++++++
 backend/src/ApppBundle/Controller/TC/Get.php                                       |  2 +-
 backend/src/ApppBundle/Controller/TC/GetList.php                                   |  2 +-
 backend/src/ApppBundle/Controller/TC/Post.php                                      |  2 +-
 backend/src/ApppBundle/Controller/TC/Put.php                                       | 57 +++++++++++++++++++++++++++++++++++++++++++++++++++++++++
 backend/src/ApppBundle/Entity/GlobalFileList.php                                   |  4 ++--
 backend/src/ApppBundle/Entity/SrcAll.php                                           | 57 +++++++++++++++++++++++++++++++++++++++++++--------------
 backend/src/ApppBundle/Entity/TC.php                                               | 57 +++++++++++++++++++++++++++++++++++++++++++--------------
 backend/src/ApppBundle/Resources/config/doctrine/SrcAll.orm.yml                    |  5 ++++-
:...skipping...
 appp-configs/appp/pawel/app-appp-tech/appp.yml                                     |  2 +-
 backend/app/DoctrineMigrations/Version20200930084310.php                           | 39 ---------------------------------------
 backend/app/DoctrineMigrations/Version20200930090608.php                           | 37 -------------------------------------
 backend/app/DoctrineMigrations/Version20200930110322.php                           | 39 ---------------------------------------
 backend/src/ApppBundle/Controller/SrcAll/Get.php                                   |  2 +-
 backend/src/ApppBundle/Controller/SrcAll/GetList.php                               |  2 +-
 backend/src/ApppBundle/Controller/SrcAll/Post.php                                  |  2 +-
 backend/src/ApppBundle/Controller/SrcAll/Put.php                                   | 57 +++++++++++++++++++++++++++++++++++++++++++++++++++++++++
 backend/src/ApppBundle/Controller/TC/Get.php                                       |  2 +-
 backend/src/ApppBundle/Controller/TC/GetList.php                                   |  2 +-
 backend/src/ApppBundle/Controller/TC/Post.php                                      |  2 +-
 backend/src/ApppBundle/Controller/TC/Put.php                                       | 57 +++++++++++++++++++++++++++++++++++++++++++++++++++++++++
 backend/src/ApppBundle/Entity/GlobalFileList.php                                   |  4 ++--
 backend/src/ApppBundle/Entity/SrcAll.php                                           | 57 +++++++++++++++++++++++++++++++++++++++++++--------------
 backend/src/ApppBundle/Entity/TC.php                                               | 57 +++++++++++++++++++++++++++++++++++++++++++--------------
 backend/src/ApppBundle/Resources/config/doctrine/SrcAll.orm.yml                    |  5 ++++-
 backend/src/ApppBundle/Resources/config/doctrine/TC.orm.yml                        |  5 ++++-
:...skipping...
 appp-configs/appp/pawel/app-appp-tech/appp.yml                                     |  2 +-
 backend/app/DoctrineMigrations/Version20200930084310.php                           | 39 ---------------------------------------
 backend/app/DoctrineMigrations/Version20200930090608.php                           | 37 -------------------------------------
 backend/app/DoctrineMigrations/Version20200930110322.php                           | 39 ---------------------------------------
 backend/src/ApppBundle/Controller/SrcAll/Get.php                                   |  2 +-
 backend/src/ApppBundle/Controller/SrcAll/GetList.php                               |  2 +-
 backend/src/ApppBundle/Controller/SrcAll/Post.php                                  |  2 +-
 backend/src/ApppBundle/Controller/SrcAll/Put.php                                   | 57 +++++++++++++++++++++++++++++++++++++++++++++++++++++++++
 backend/src/ApppBundle/Controller/TC/Get.php                                       |  2 +-
 backend/src/ApppBundle/Controller/TC/GetList.php                                   |  2 +-
 backend/src/ApppBundle/Controller/TC/Post.php                                      |  2 +-
 backend/src/ApppBundle/Controller/TC/Put.php                                       | 57 +++++++++++++++++++++++++++++++++++++++++++++++++++++++++
 backend/src/AiqaBundle/Entity/GlobalFileList.php                                   |  4 ++--
 backend/src/AiqaBundle/Entity/SrcAll.php                                           | 57 +++++++++++++++++++++++++++++++++++++++++++--------------
 backend/src/ApppBundle/Entity/TC.php                                               | 57 +++++++++++++++++++++++++++++++++++++++++++--------------
 backend/src/ApppBundle/Resources/config/doctrine/SrcAll.orm.yml                    |  5 ++++-
 backend/src/ApppBundle/Resources/config/doctrine/TC.orm.yml                        |  5 ++++-
 backend/src/ApppBundle/RestApi/SrcAll/GetList.php                                  |  2 +-
:...skipping...
 appp-configs/appp/pawel/app-appp-tech/appp.yml                                     |  2 +-
 backend/app/DoctrineMigrations/Version20200930084310.php                           | 39 ---------------------------------------
 backend/app/DoctrineMigrations/Version20200930090608.php                           | 37 -------------------------------------
 backend/app/DoctrineMigrations/Version20200930110322.php                           | 39 ---------------------------------------
 backend/src/ApppBundle/Controller/SrcAll/Get.php                                   |  2 +-
 backend/src/ApppBundle/Controller/SrcAll/GetList.php                               |  2 +-
 backend/src/ApppBundle/Controller/SrcAll/Post.php                                  |  2 +-
 backend/src/ApppBundle/Controller/SrcAll/Put.php                                   | 57 +++++++++++++++++++++++++++++++++++++++++++++++++++++++++
 backend/src/ApppBundle/Controller/TC/Get.php                                       |  2 +-
 backend/src/ApppBundle/Controller/TC/GetList.php                                   |  2 +-
 backend/src/ApppBundle/Controller/TC/Post.php                                      |  2 +-
 backend/src/ApppBundle/Controller/TC/Put.php                                       | 57 +++++++++++++++++++++++++++++++++++++++++++++++++++++++++
 backend/src/ApppBundle/Entity/GlobalFileList.php                                   |  4 ++--
 backend/src/ApppBundle/Entity/SrcAll.php                                           | 57 +++++++++++++++++++++++++++++++++++++++++++--------------
 backend/src/ApppBundle/Entity/TC.php                                               | 57 +++++++++++++++++++++++++++++++++++++++++++--------------
 backend/src/ApppBundle/Resources/config/doctrine/SrcAll.orm.yml                    |  5 ++++-
 backend/src/ApppBundle/Resources/config/doctrine/TC.orm.yml                        |  5 ++++-
 backend/src/ApppBundle/RestApi/SrcAll/GetList.php                                  |  2 +-
 backend/src/ApppBundle/RestApi/SrcAll/Post.php                                     | 40 +++++++++++++++++++++++++---------------
:...skipping...
 appp-configs/appp/pawel/app-appp-tech/appp.yml                                     |  2 +-
 backend/app/DoctrineMigrations/Version20200930084310.php                           | 39 ---------------------------------------
 backend/app/DoctrineMigrations/Version20200930090608.php                           | 37 -------------------------------------
 backend/app/DoctrineMigrations/Version20200930110322.php                           | 39 ---------------------------------------
 backend/src/ApppBundle/Controller/SrcAll/Get.php                                   |  2 +-
 backend/src/ApppBundle/Controller/SrcAll/GetList.php                               |  2 +-
 backend/src/ApppBundle/Controller/SrcAll/Post.php                                  |  2 +-
 backend/src/ApppBundle/Controller/SrcAll/Put.php                                   | 57 +++++++++++++++++++++++++++++++++++++++++++++++++++++++++
 backend/src/ApppBundle/Controller/TC/Get.php                                       |  2 +-
 backend/src/ApppBundle/Controller/TC/GetList.php                                   |  2 +-
 backend/src/ApppBundle/Controller/TC/Post.php                                      |  2 +-
 backend/src/AiqaBundle/Controller/TC/Put.php                                       | 57 +++++++++++++++++++++++++++++++++++++++++++++++++++++++++
 backend/src/ApppBundle/Entity/GlobalFileList.php                                   |  4 ++--
 backend/src/ApppBundle/Entity/SrcAll.php                                           | 57 +++++++++++++++++++++++++++++++++++++++++++--------------
 backend/src/ApppBundle/Entity/TC.php                                               | 57 +++++++++++++++++++++++++++++++++++++++++++--------------
 backend/src/ApppBundle/Resources/config/doctrine/SrcAll.orm.yml                    |  5 ++++-
 backend/src/ApppBundle/Resources/config/doctrine/TC.orm.yml                        |  5 ++++-
 backend/src/ApppBundle/RestApi/SrcAll/GetList.php                                  |  2 +-
 backend/src/ApppBundle/RestApi/SrcAll/Post.php                                     | 40 +++++++++++++++++++++++++---------------
 backend/src/ApppBundle/RestApi/SrcAll/Put.php                                      | 64 ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
:...skipping...
 appp-configs/appp/pawel/app-appp-tech/appp.yml                                     |  2 +-
 backend/app/DoctrineMigrations/Version20200930084310.php                           | 39 ---------------------------------------
 backend/app/DoctrineMigrations/Version20200930090608.php                           | 37 -------------------------------------
 backend/app/DoctrineMigrations/Version20200930110322.php                           | 39 ---------------------------------------
 backend/src/ApppBundle/Controller/SrcAll/Get.php                                   |  2 +-
 backend/src/ApppBundle/Controller/SrcAll/GetList.php                               |  2 +-
 backend/src/ApppBundle/Controller/SrcAll/Post.php                                  |  2 +-
 backend/src/ApppBundle/Controller/SrcAll/Put.php                                   | 57 +++++++++++++++++++++++++++++++++++++++++++++++++++++++++
 backend/src/ApppBundle/Controller/TC/Get.php                                       |  2 +-
 backend/src/ApppBundle/Controller/TC/GetList.php                                   |  2 +-
 backend/src/ApppBundle/Controller/TC/Post.php                                      |  2 +-
 backend/src/ApppBundle/Controller/TC/Put.php                                       | 57 +++++++++++++++++++++++++++++++++++++++++++++++++++++++++
 backend/src/ApppBundle/Entity/GlobalFileList.php                                   |  4 ++--
 backend/src/ApppBundle/Entity/SrcAll.php                                           | 57 +++++++++++++++++++++++++++++++++++++++++++--------------
 backend/src/ApppBundle/Entity/TC.php                                               | 57 +++++++++++++++++++++++++++++++++++++++++++--------------
 backend/src/ApppBundle/Resources/config/doctrine/SrcAll.orm.yml                    |  5 ++++-
 backend/src/ApppBundle/Resources/config/doctrine/TC.orm.yml                        |  5 ++++-
 backend/src/ApppBundle/RestApi/SrcAll/GetList.php                                  |  2 +-
 backend/src/ApppBundle/RestApi/SrcAll/Post.php                                     | 40 +++++++++++++++++++++++++---------------
 backend/src/ApppBundle/RestApi/SrcAll/Put.php                                      | 64 ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
 backend/src/ApppBundle/RestApi/TC/GetList.php                                      |  2 +-
 backend/src/ApppBundle/RestApi/TC/Post.php                                         | 41 ++++++++++++++++++++++++++---------------
:...skipping...
 appp-configs/appp/pawel/app-appp-tech/appp.yml                                     |  2 +-
 backend/app/DoctrineMigrations/Version20200930084310.php                           | 39 ---------------------------------------
 backend/app/DoctrineMigrations/Version20200930090608.php                           | 37 -------------------------------------
 backend/app/DoctrineMigrations/Version20200930110322.php                           | 39 ---------------------------------------
 backend/src/ApppBundle/Controller/SrcAll/Get.php                                   |  2 +-
 backend/src/ApppBundle/Controller/SrcAll/GetList.php                               |  2 +-
 backend/src/ApppBundle/Controller/SrcAll/Post.php                                  |  2 +-
 backend/src/ApppBundle/Controller/SrcAll/Put.php                                   | 57 +++++++++++++++++++++++++++++++++++++++++++++++++++++++++
 backend/src/ApppBundle/Controller/TC/Get.php                                       |  2 +-
 backend/src/ApppBundle/Controller/TC/GetList.php                                   |  2 +-
 backend/src/ApppBundle/Controller/TC/Post.php                                      |  2 +-
 backend/src/ApppBundle/Controller/TC/Put.php                                       | 57 +++++++++++++++++++++++++++++++++++++++++++++++++++++++++
 backend/src/ApppBundle/Entity/GlobalFileList.php                                   |  4 ++--
 backend/src/ApppBundle/Entity/SrcAll.php                                           | 57 +++++++++++++++++++++++++++++++++++++++++++--------------
 backend/src/ApppBundle/Entity/TC.php                                               | 57 +++++++++++++++++++++++++++++++++++++++++++--------------
 backend/src/ApppBundle/Resources/config/doctrine/SrcAll.orm.yml                    |  5 ++++-
 backend/src/ApppBundle/Resources/config/doctrine/TC.orm.yml                        |  5 ++++-
 backend/src/ApppBundle/RestApi/SrcAll/GetList.php                                  |  2 +-
 backend/src/ApppBundle/RestApi/SrcAll/Post.php                                     | 40 +++++++++++++++++++++++++---------------
 backend/src/ApppBundle/RestApi/SrcAll/Put.php                                      | 64 ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
 backend/src/ApppBundle/RestApi/TC/GetList.php                                      |  2 +-
 backend/src/ApppBundle/RestApi/TC/Post.php                                         | 41 ++++++++++++++++++++++++++---------------
 backend/src/ApppBundle/RestApi/TC/Put.php                                          | 64 ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
 backend/src/ApppBundle/Tools/Utilities/FailedTests/MergeStrategyUnifier.php        | 43 ++++---------------------------------------
:...skipping...
 appp-configs/appp/pawel/app-appp-tech/appp.yml                                     |  2 +-
 backend/app/DoctrineMigrations/Version20200930084310.php                           | 39 ---------------------------------------
 backend/app/DoctrineMigrations/Version20200930090608.php                           | 37 -------------------------------------
 backend/app/DoctrineMigrations/Version20200930110322.php                           | 39 ---------------------------------------
 backend/src/ApppBundle/Controller/SrcAll/Get.php                                   |  2 +-
 backend/src/ApppBundle/Controller/SrcAll/GetList.php                               |  2 +-
 backend/src/ApppBundle/Controller/SrcAll/Post.php                                  |  2 +-
 backend/src/ApppBundle/Controller/SrcAll/Put.php                                   | 57 +++++++++++++++++++++++++++++++++++++++++++++++++++++++++
 backend/src/ApppBundle/Controller/TC/Get.php                                       |  2 +-
 backend/src/ApppBundle/Controller/TC/GetList.php                                   |  2 +-
 backend/src/ApppBundle/Controller/TC/Post.php                                      |  2 +-
 backend/src/ApppBundle/Controller/TC/Put.php                                       | 57 +++++++++++++++++++++++++++++++++++++++++++++++++++++++++
 backend/src/ApppBundle/Entity/GlobalFileList.php                                   |  4 ++--
 backend/src/ApppBundle/Entity/SrcAll.php                                           | 57 +++++++++++++++++++++++++++++++++++++++++++--------------
 backend/src/ApppBundle/Entity/TC.php                                               | 57 +++++++++++++++++++++++++++++++++++++++++++--------------
 backend/src/ApppBundle/Resources/config/doctrine/SrcAll.orm.yml                    |  5 ++++-
 backend/src/ApppBundle/Resources/config/doctrine/TC.orm.yml                        |  5 ++++-
 backend/src/ApppBundle/RestApi/SrcAll/GetList.php                                  |  2 +-
 backend/src/ApppBundle/RestApi/SrcAll/Post.php                                     | 40 +++++++++++++++++++++++++---------------
 backend/src/ApppBundle/RestApi/SrcAll/Put.php                                      | 64 ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
 backend/src/ApppBundle/RestApi/TC/GetList.php                                      |  2 +-
 backend/src/ApppBundle/RestApi/TC/Post.php                                         | 41 ++++++++++++++++++++++++++---------------
 backend/src/ApppBundle/RestApi/TC/Put.php                                          | 64 ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
 backend/src/ApppBundle/Tools/Utilities/FailedTests/MergeStrategyUnifier.php        | 43 ++++---------------------------------------
 behat/_ci_appp.sh                                                                  |  8 ++++----
:...skipping...
 appp-configs/appp/pawel/app-appp-tech/appp.yml                                     |  2 +-
 backend/app/DoctrineMigrations/Version20200930084310.php                           | 39 ---------------------------------------
 backend/app/DoctrineMigrations/Version20200930090608.php                           | 37 -------------------------------------
 backend/app/DoctrineMigrations/Version20200930110322.php                           | 39 ---------------------------------------
 backend/src/ApppBundle/Controller/SrcAll/Get.php                                   |  2 +-
 backend/src/ApppBundle/Controller/SrcAll/GetList.php                               |  2 +-
 backend/src/ApppBundle/Controller/SrcAll/Post.php                                  |  2 +-
 backend/src/ApppBundle/Controller/SrcAll/Put.php                                   | 57 +++++++++++++++++++++++++++++++++++++++++++++++++++++++++
 backend/src/ApppBundle/Controller/TC/Get.php                                       |  2 +-
 backend/src/ApppBundle/Controller/TC/GetList.php                                   |  2 +-
 backend/src/ApppBundle/Controller/TC/Post.php                                      |  2 +-
 backend/src/ApppBundle/Controller/TC/Put.php                                       | 57 +++++++++++++++++++++++++++++++++++++++++++++++++++++++++
 backend/src/ApppBundle/Entity/GlobalFileList.php                                   |  4 ++--
 backend/src/ApppBundle/Entity/SrcAll.php                                           | 57 +++++++++++++++++++++++++++++++++++++++++++--------------
 backend/src/ApppBundle/Entity/TC.php                                               | 57 +++++++++++++++++++++++++++++++++++++++++++--------------
 backend/src/ApppBundle/Resources/config/doctrine/SrcAll.orm.yml                    |  5 ++++-
 backend/src/ApppBundle/Resources/config/doctrine/TC.orm.yml                        |  5 ++++-
 backend/src/ApppBundle/RestApi/SrcAll/GetList.php                                  |  2 +-
 backend/src/ApppBundle/RestApi/SrcAll/Post.php                                     | 40 +++++++++++++++++++++++++---------------
 backend/src/ApppBundle/RestApi/SrcAll/Put.php                                      | 64 ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
 backend/src/ApppBundle/RestApi/TC/GetList.php                                      |  2 +-
 backend/src/ApppBundle/RestApi/TC/Post.php                                         | 41 ++++++++++++++++++++++++++---------------
 backend/src/ApppBundle/RestApi/TC/Put.php                                          | 64 ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
 backend/src/ApppBundle/Tools/Utilities/FailedTests/MergeStrategyUnifier.php        | 43 ++++---------------------------------------
 behat/_ci_aiqa.sh                                                                  |  8 ++++----
 behat/features/api/build/prediction/get_tests_strategies_for_empty_project.feature |  2 +-
:...skipping...
 appp-configs/appp/pawel/app-appp-tech/appp.yml                                     |  2 +-
 backend/app/DoctrineMigrations/Version20200930084310.php                           | 39 ---------------------------------------
 backend/app/DoctrineMigrations/Version20200930090608.php                           | 37 -------------------------------------
 backend/app/DoctrineMigrations/Version20200930110322.php                           | 39 ---------------------------------------
 backend/src/ApppBundle/Controller/SrcAll/Get.php                                   |  2 +-
 backend/src/ApppBundle/Controller/SrcAll/GetList.php                               |  2 +-
 backend/src/ApppBundle/Controller/SrcAll/Post.php                                  |  2 +-
 backend/src/ApppBundle/Controller/SrcAll/Put.php                                   | 57 +++++++++++++++++++++++++++++++++++++++++++++++++++++++++
 backend/src/ApppBundle/Controller/TC/Get.php                                       |  2 +-
 backend/src/ApppBundle/Controller/TC/GetList.php                                   |  2 +-
 backend/src/ApppBundle/Controller/TC/Post.php                                      |  2 +-
 backend/src/ApppBundle/Controller/TC/Put.php                                       | 57 +++++++++++++++++++++++++++++++++++++++++++++++++++++++++
 backend/src/ApppBundle/Entity/GlobalFileList.php                                   |  4 ++--
 backend/src/ApppBundle/Entity/SrcAll.php                                           | 57 +++++++++++++++++++++++++++++++++++++++++++--------------
 backend/src/ApppBundle/Entity/TC.php                                               | 57 +++++++++++++++++++++++++++++++++++++++++++--------------
 backend/src/ApppBundle/Resources/config/doctrine/SrcAll.orm.yml                    |  5 ++++-
 backend/src/ApppBundle/Resources/config/doctrine/TC.orm.yml                        |  5 ++++-
 backend/src/ApppBundle/RestApi/SrcAll/GetList.php                                  |  2 +-
 backend/src/ApppBundle/RestApi/SrcAll/Post.php                                     | 40 +++++++++++++++++++++++++---------------
 backend/src/ApppBundle/RestApi/SrcAll/Put.php                                      | 64 ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
 backend/src/ApppBundle/RestApi/TC/GetList.php                                      |  2 +-
 backend/src/ApppBundle/RestApi/TC/Post.php                                         | 41 ++++++++++++++++++++++++++---------------
 backend/src/ApppBundle/RestApi/TC/Put.php                                          | 64 ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
 backend/src/ApppBundle/Tools/Utilities/FailedTests/MergeStrategyUnifier.php        | 43 ++++---------------------------------------
 behat/_ci_appp.sh                                                                  |  8 ++++----
 behat/features/api/build/prediction/get_tests_strategies_for_empty_project.feature |  2 +-
 behat/features/api/build/prediction/get_tests_strategy_from_project_config.feature |  2 +-
:...skipping...
 appp-configs/appp/pawel/app-appp-tech/appp.yml                                     |  2 +-
 backend/app/DoctrineMigrations/Version20200930084310.php                           | 39 ---------------------------------------
 backend/app/DoctrineMigrations/Version20200930090608.php                           | 37 -------------------------------------
 backend/app/DoctrineMigrations/Version20200930110322.php                           | 39 ---------------------------------------
 backend/src/ApppBundle/Controller/SrcAll/Get.php                                   |  2 +-
 backend/src/ApppBundle/Controller/SrcAll/GetList.php                               |  2 +-
 backend/src/ApppBundle/Controller/SrcAll/Post.php                                  |  2 +-
 backend/src/ApppBundle/Controller/SrcAll/Put.php                                   | 57 +++++++++++++++++++++++++++++++++++++++++++++++++++++++++
 backend/src/ApppBundle/Controller/TC/Get.php                                       |  2 +-
 backend/src/ApppBundle/Controller/TC/GetList.php                                   |  2 +-
 backend/src/ApppBundle/Controller/TC/Post.php                                      |  2 +-
 backend/src/ApppBundle/Controller/TC/Put.php                                       | 57 +++++++++++++++++++++++++++++++++++++++++++++++++++++++++
 backend/src/ApppBundle/Entity/GlobalFileList.php                                   |  4 ++--
 backend/src/ApppBundle/Entity/SrcAll.php                                           | 57 +++++++++++++++++++++++++++++++++++++++++++--------------
 backend/src/ApppBundle/Entity/TC.php                                               | 57 +++++++++++++++++++++++++++++++++++++++++++--------------
 backend/src/ApppBundle/Resources/config/doctrine/SrcAll.orm.yml                    |  5 ++++-
 backend/src/ApppBundle/Resources/config/doctrine/TC.orm.yml                        |  5 ++++-
 backend/src/ApppBundle/RestApi/SrcAll/GetList.php                                  |  2 +-
 backend/src/ApppBundle/RestApi/SrcAll/Post.php                                     | 40 +++++++++++++++++++++++++---------------
 backend/src/ApppBundle/RestApi/SrcAll/Put.php                                      | 64 ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
 backend/src/ApppBundle/RestApi/TC/GetList.php                                      |  2 +-
 backend/src/ApppBundle/RestApi/TC/Post.php                                         | 41 ++++++++++++++++++++++++++---------------
 backend/src/ApppBundle/RestApi/TC/Put.php                                          | 64 ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
 backend/src/ApppBundle/Tools/Utilities/FailedTests/MergeStrategyUnifier.php        | 43 ++++---------------------------------------
 behat/_ci_appp.sh                                                                  |  8 ++++----
 behat/features/api/build/prediction/get_tests_strategies_for_empty_project.feature |  2 +-
 behat/features/api/build/prediction/get_tests_strategy_from_project_config.feature |  2 +-
 behat/features/api/build/prediction/strategy_changedTests.feature                  |  8 ++++----
:...skipping...
 appp-configs/appp/pawel/app-appp-tech/appp.yml                                     |  2 +-
 backend/app/DoctrineMigrations/Version20200930084310.php                           | 39 ---------------------------------------
 backend/app/DoctrineMigrations/Version20200930090608.php                           | 37 -------------------------------------
 backend/app/DoctrineMigrations/Version20200930110322.php                           | 39 ---------------------------------------
 backend/src/ApppBundle/Controller/SrcAll/Get.php                                   |  2 +-
 backend/src/ApppBundle/Controller/SrcAll/GetList.php                               |  2 +-
 backend/src/ApppBundle/Controller/SrcAll/Post.php                                  |  2 +-
 backend/src/ApppBundle/Controller/SrcAll/Put.php                                   | 57 +++++++++++++++++++++++++++++++++++++++++++++++++++++++++
 backend/src/ApppBundle/Controller/TC/Get.php                                       |  2 +-
 backend/src/ApppBundle/Controller/TC/GetList.php                                   |  2 +-
 backend/src/ApppBundle/Controller/TC/Post.php                                      |  2 +-
 backend/src/ApppBundle/Controller/TC/Put.php                                       | 57 +++++++++++++++++++++++++++++++++++++++++++++++++++++++++
 backend/src/ApppBundle/Entity/GlobalFileList.php                                   |  4 ++--
 backend/src/ApppBundle/Entity/SrcAll.php                                           | 57 +++++++++++++++++++++++++++++++++++++++++++--------------
 backend/src/ApppBundle/Entity/TC.php                                               | 57 +++++++++++++++++++++++++++++++++++++++++++--------------
 backend/src/ApppBundle/Resources/config/doctrine/SrcAll.orm.yml                    |  5 ++++-
 backend/src/ApppBundle/Resources/config/doctrine/TC.orm.yml                        |  5 ++++-
 backend/src/ApppBundle/RestApi/SrcAll/GetList.php                                  |  2 +-
 backend/src/ApppBundle/RestApi/SrcAll/Post.php                                     | 40 +++++++++++++++++++++++++---------------
 backend/src/ApppBundle/RestApi/SrcAll/Put.php                                      | 64 ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
 backend/src/ApppBundle/RestApi/TC/GetList.php                                      |  2 +-
 backend/src/ApppBundle/RestApi/TC/Post.php                                         | 41 ++++++++++++++++++++++++++---------------
 backend/src/ApppBundle/RestApi/TC/Put.php                                          | 64 ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
 backend/src/ApppBundle/Tools/Utilities/FailedTests/MergeStrategyUnifier.php        | 43 ++++---------------------------------------
 behat/_ci_appp.sh                                                                  |  8 ++++----
 behat/features/api/build/prediction/get_tests_strategies_for_empty_project.feature |  2 +-
 behat/features/api/build/prediction/get_tests_strategy_from_project_config.feature |  2 +-
 behat/features/api/build/prediction/strategy_changedTests.feature                  |  8 ++++----
 behat/features/api/build/prediction/strategy_full_set.feature                      | 10 +++++-----
 behat/features/api/build/prediction/strategy_map.feature                           |  8 ++++----
 behat/features/api/build/prediction/strategy_none.feature                          |  8 ++++----
 behat/features/api/build/prediction/strategy_one.feature                           |  8 ++++----
 behat/features/api/build/prediction/strategy_random1.feature                       |  6 +++---
 behat/features/api/build/prediction/strategy_random3.feature                       | 10 +++++-----
 behat/features/api/build/prediction/strategy_smoke.feature                         |  8 ++++----
 behat/features/api/build/prediction/strategy_srcTags.feature                       |  8 ++++----
 behat/features/api/build/prediction/strategy_tags.feature                          |  6 +++---
 behat/features/api/project/prediction/get_tests_strategies_empty_project.feature   |  8 ++++----
 behat/features/api/project/prediction/role_only_prediction.feature                 |  2 +-
 behat/features/api/project/prediction/strategy_none.feature                        |  6 +++---
 behat/features/api/project/prediction/strategy_random.feature                      | 20 ++++++++++----------
 behat/features/api/project/prediction/strategy_random1.feature                     |  8 ++++----
 behat/features/api/project/prediction/strategy_random3.feature                     |  8 ++++----
 behat/features/api/project/prediction/strategy_srcTags.feature                     |  6 +++---
 behat/features/api/project/prediction/strategy_tags.feature                        | 28 ++++++++++++++--------------
 behat/features/api/srcAll/authorization/admin_put.feature                          | 57 +++++++++++++++++++++++++++++++++++++++++++++++++++++++++
 behat/features/api/srcAll/authorization/unauthorized_put.feature                   | 51 +++++++++++++++++++++++++++++++++++++++++++++++++++
 behat/features/api/srcAll/crud/put.feature                                         | 58 ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
 behat/features/api/srcAll/miscellaneous/put_wrong_path.feature                     | 57 +++++++++++++++++++++++++++++++++++++++++++++++++++++++++
 behat/features/api/srcAll/miscellaneous/use_token_to_put.feature                   | 51 +++++++++++++++++++++++++++++++++++++++++++++++++++
 behat/features/api/tc/authorization/admin_put.feature                              | 41 +++++++++++++++++++++++++++++++++++++++++
 behat/features/api/tc/authorization/unauthorized_put.feature                       | 36 ++++++++++++++++++++++++++++++++++++
 behat/features/api/tc/crud/put.feature                                             | 58 ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
 behat/features/api/tc/miscellaneous/create_repeated_data.feature                   | 33 +++++++++++++++++++++++++++++++++
 behat/features/api/tc/miscellaneous/put_wrong_path.feature                         | 42 ++++++++++++++++++++++++++++++++++++++++++
 behat/features/api/tc/miscellaneous/use_token_to_put.feature                       | 36 ++++++++++++++++++++++++++++++++++++
 documentation/redoc/response/srcAll.json                                           | 14 +++-----------
 documentation/redoc/response/srcAll_array.json                                     | 14 +++-----------
 documentation/redoc/response/tc.json                                               | 14 +++-----------
 documentation/redoc/response/tc_array.json                                         | 14 +++-----------
 json-schema/response/srcAll.json                                                   | 14 +++-----------
 json-schema/response/srcAll_array.json                                             | 14 +++-----------
 json-schema/response/tc.json                                                       | 15 ++++-----------
 json-schema/response/tc_array.json                                                 | 15 ++++-----------
 json-schema/response/tests_prediction.json                                         | 20 --------------------
 mysqldump/appp.sql                                                                 | 39 ++++++++++++++++++++++++---------------
 66 files changed, 1061 insertions(+), 437 deletions(-)
@TomAdmz
Copy link

TomAdmz commented Apr 30, 2021

Also having this issue @pawelWritesCode, were you ever able to figure out the root cause?

@egibert
Copy link

egibert commented May 4, 2021 •
edited

For anyone having the same issue, this happens when the file path that you made changes contains more than 65 characters, heightOfHistogram becomes negative. And then you are dividing a positive number with a negative number and it panics.

go-git/plumbing/object/patch.go

Line 275 in bf3471d

heightOfHistogram := lineLength - totalTextArea

@github-actions GitHub Actions
Copy link

To help us keep things tidy and focus on the active tasks, we've introduced a stale bot to spot issues/PRs that haven't had any activity in a while.

This particular issue hasn't had any updates or activity in the past 90 days, so it's been labeled as 'stale'. If it remains inactive for the next 30 days, it'll be automatically closed.

We understand everyone's busy, but if this issue is still important to you, please feel free to add a comment or make an update to keep it active.

Thanks for your understanding and cooperation!

@github-actions github-actions bot added the stale Issues/PRs that are marked for closure due to inactivity label Dec 13, 2023
@pjbgf pjbgf added bug Something isn't working good first issue Good for newcomers help wanted Extra attention is needed and removed stale Issues/PRs that are marked for closure due to inactivity labels Dec 14, 2023
nodivbyzero added a commit to nodivbyzero/go-git that referenced this issue Dec 15, 2023
@nodivbyzero
plumbing: no panic in printStat function. Fixes go-git#177
0e1a027
@nodivbyzero nodivbyzero mentioned this issue Dec 15, 2023
Merged
@pjbgf pjbgf closed this as completed in d058d58 Mar 11, 2024
pjbgf added a commit that referenced this issue Mar 11, 2024
@pjbgf
Merge pull request #971 from nodivbyzero/fix-177-diff-print-file-stats
f4f1a87 
plumbing: no panic in printStats function. Fixes #177
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug Something isn't working good first issue Good for newcomers help wanted Extra attention is needed
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
4 participants
@egibert @pjbgf @TomAdmz @pawelWritesCode