Page MenuHomePhabricator

Merge #13149: Handle unsuccessful fseek(...):s

Authored by nakihito on Sep 6 2019, 18:21.



20ce5af Print a log message if we fail to shrink the debug log file (practicalswift)
29c9bdc Handle unsuccessful fseek(...):s (practicalswift)

Pull request description:

Handle unsuccessful `fseek(...)`:s.

**Note to reviewers:** What is the most appropriate course of actions for each of these unsuccessful `fseek(...)`:s?

Tree-SHA512: 5b3d82dbdd15d434d3f08dcb4df62888da4df8541d2586f56a4e529083005f6782c39e10645acd1ec403da83061bbfd8dbf2dddc66e09268d410ad0918c61876

Backport of Core PR13149

Test Plan
make check

Diff Detail

rABC Bitcoin ABC
Automatic diff as part of commit; lint not applicable.
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

nakihito created this revision.Sep 6 2019, 18:21
Owners added a reviewer: Restricted Owners Package.Sep 6 2019, 18:21
Herald added a reviewer: Restricted Project. · View Herald TranscriptSep 6 2019, 18:21
deadalnix requested changes to this revision.Sep 8 2019, 12:10

Mention PR13149

This revision now requires changes to proceed.Sep 8 2019, 12:10
nakihito requested review of this revision.Sep 9 2019, 17:06
nakihito edited the summary of this revision. (Show Details)
Fabien accepted this revision.Sep 10 2019, 06:37
deadalnix accepted this revision.Sep 14 2019, 08:32
This revision is now accepted and ready to land.Sep 14 2019, 08:32
This revision was landed with ongoing or failed builds.Sep 16 2019, 23:45
This revision was automatically updated to reflect the committed changes.