search

lilydjwg / ncdu-diff

ncdu fork that can compare and diff results
MIT License
27 stars 4 forks source link

patch会导致统计出问题 #2

Closed Alkaid-Benetnash closed 6 years ago

Alkaid-Benetnash commented 6 years ago

patch之后会有这种结果 image 未patch是这样的 image 下方最后统计的总量也有问题。

lilydjwg commented 6 years ago

原来是被我改出来的 bug 啊……

lilydjwg commented 6 years ago

C 语言真是糟糕，abs、labs、llabs……还有 fabs、fabsf、fabsl……而且没有警告 (╯‵□′)╯︵┻━┻

Alkaid-Benetnash commented 6 years ago

int64_t不应该选llabs吗？labs的参数类型long是int32_t吧？

lilydjwg commented 6 years ago

咦，我这里的 long 和 long long 好像是一样大？

Alkaid-Benetnash commented 6 years ago

应该和32还是64以及编译器都有关，都是坑不管了。 新patch在我这里是正常的， thanks!