This is an archive of the discontinued Mercurial Phabricator instance.

revlog: use %d to format int instead of %lu (issue6565)
AbandonedPublic

Authored by av6 on Tue, May 17, 2:14 PM.

Details

Reviewers
None
Group Reviewers
hg-reviewers
Summary

The issue says gcc warns that the data types don't match. I couldn't reproduce
the warning locally for some reason, but this patch shouldn't break things.
Maybe %lu was simply a copy-paste error from 6b1eae313b2f
(https://phab.mercurial-scm.org/D10625).

Diff Detail

Repository
rHG Mercurial
Branch
default
Lint
No Linters Available
Unit
No Unit Test Coverage

Event Timeline

av6 created this revision.Tue, May 17, 2:14 PM

I think there are two issues here. The format string in line 500 doesn't match 492 and the format string in 494 doesn't either.