Mercurial > public > mercurial-scm > hg-stable
comparison mercurial/debugcommands.py @ 37285:d4e62df1c73d
debugcommands: drop offset and length from debugindex by default
These fields are an implementation detail of revlog storage. As
such, they are not part of the generic storage "index" interface
and shouldn't be displayed by default.
Because we don't have another way to display these fields, we've
retained support for printing these fields via --verbose.
Yes, I know we should probably be doing all this formatting using
modern formatting/templater APIs. I didn't feel like scope
bloating this patch.
Differential Revision: https://phab.mercurial-scm.org/D3028
author | Gregory Szorc <gregory.szorc@gmail.com> |
---|---|
date | Mon, 02 Apr 2018 16:47:53 -0700 |
parents | 009d0283de5f |
children | 9bfcbe4f4745 |
comparison
equal
deleted
inserted
replaced
37284:009d0283de5f | 37285:d4e62df1c73d |
---|---|
1069 for i in r: | 1069 for i in r: |
1070 idlen = len(shortfn(r.node(i))) | 1070 idlen = len(shortfn(r.node(i))) |
1071 break | 1071 break |
1072 | 1072 |
1073 if format == 0: | 1073 if format == 0: |
1074 ui.write((" rev offset length linkrev" | 1074 if ui.verbose: |
1075 " %s %s p2\n") % ("nodeid".ljust(idlen), "p1".ljust(idlen))) | 1075 ui.write((" rev offset length linkrev" |
1076 " %s %s p2\n") % ("nodeid".ljust(idlen), | |
1077 "p1".ljust(idlen))) | |
1078 else: | |
1079 ui.write((" rev linkrev %s %s p2\n") % ( | |
1080 "nodeid".ljust(idlen), "p1".ljust(idlen))) | |
1076 elif format == 1: | 1081 elif format == 1: |
1077 ui.write((" rev flag offset length size link p1 p2" | 1082 if ui.verbose: |
1078 " %s\n") % "nodeid".rjust(idlen)) | 1083 ui.write((" rev flag offset length size link p1" |
1084 " p2 %s\n") % "nodeid".rjust(idlen)) | |
1085 else: | |
1086 ui.write((" rev flag size link p1 p2 %s\n") % | |
1087 "nodeid".rjust(idlen)) | |
1079 | 1088 |
1080 for i in r: | 1089 for i in r: |
1081 node = r.node(i) | 1090 node = r.node(i) |
1082 if format == 0: | 1091 if format == 0: |
1083 try: | 1092 try: |
1084 pp = r.parents(node) | 1093 pp = r.parents(node) |
1085 except Exception: | 1094 except Exception: |
1086 pp = [nullid, nullid] | 1095 pp = [nullid, nullid] |
1087 ui.write("% 6d % 9d % 7d % 7d %s %s %s\n" % ( | 1096 if ui.verbose: |
1088 i, r.start(i), r.length(i), r.linkrev(i), | 1097 ui.write("% 6d % 9d % 7d % 7d %s %s %s\n" % ( |
1089 shortfn(node), shortfn(pp[0]), shortfn(pp[1]))) | 1098 i, r.start(i), r.length(i), r.linkrev(i), |
1099 shortfn(node), shortfn(pp[0]), shortfn(pp[1]))) | |
1100 else: | |
1101 ui.write("% 6d % 7d %s %s %s\n" % ( | |
1102 i, r.linkrev(i), shortfn(node), shortfn(pp[0]), | |
1103 shortfn(pp[1]))) | |
1090 elif format == 1: | 1104 elif format == 1: |
1091 pr = r.parentrevs(i) | 1105 pr = r.parentrevs(i) |
1092 ui.write("% 6d %04x % 8d % 8d % 8d % 6d % 6d % 6d %s\n" % ( | 1106 if ui.verbose: |
1093 i, r.flags(i), r.start(i), r.length(i), r.rawsize(i), | 1107 ui.write("% 6d %04x % 8d % 8d % 8d % 6d % 6d % 6d %s\n" % ( |
1094 r.linkrev(i), pr[0], pr[1], shortfn(node))) | 1108 i, r.flags(i), r.start(i), r.length(i), r.rawsize(i), |
1109 r.linkrev(i), pr[0], pr[1], shortfn(node))) | |
1110 else: | |
1111 ui.write("% 6d %04x % 8d % 6d % 6d % 6d %s\n" % ( | |
1112 i, r.flags(i), r.rawsize(i), r.linkrev(i), pr[0], pr[1], | |
1113 shortfn(node))) | |
1095 | 1114 |
1096 @command('debugindexdot', cmdutil.debugrevlogopts, | 1115 @command('debugindexdot', cmdutil.debugrevlogopts, |
1097 _('-c|-m|FILE'), optionalrepo=True) | 1116 _('-c|-m|FILE'), optionalrepo=True) |
1098 def debugindexdot(ui, repo, file_=None, **opts): | 1117 def debugindexdot(ui, repo, file_=None, **opts): |
1099 """dump an index DAG as a graphviz dot file""" | 1118 """dump an index DAG as a graphviz dot file""" |