Project

General

Profile

Bug #11207

many 64-bit mdb format characters have insufficient width

Added by Robert Mustacchi 18 days ago. Updated about 1 hour ago.

Status:
Closed
Priority:
Normal
Category:
mdb - modular debugger
Start date:
Due date:
% Done:

100%

Estimated time:
Difficulty:
Medium
Tags:

Description

mdb has many 64-bit format characters that are broken with respect to their designated widths, causing recurring use of format characters to present values without intervening whitespace. In most cases, this yields an unrepresentably large number:

> -1=EE
                1844674407370955161518446744073709551615
> -1=RR
                11111111111111111111111111111111111111111111111111111111111111111111111111111111111111111111111111111111111111111111111111111111
> -1=GG
                17777777777777777777771777777777777777777777

In other cases, it's just annoying:

> -1=PP
                0xffffffffffffffff0xffffffffffffffff
> -1=pp
                0xffffffffffffffff0xffffffffffffffff
The following formats are broken:
Format character Specified width Needed width Needed field width
e 16 20 21
E 16 20 21
g 16 23 24
G 16 22 23
p 16 18 19
P 16 18 19
R 64 64 65

These should be fixed, with the field width increased accordingly.

History

#1

Updated by Robert Mustacchi about 12 hours ago

  • Subject changed from many 64-bit mdb format characters have insufficient width to many 64-bit mdb format characters have insufficient width
#2

Updated by Electric Monk about 1 hour ago

  • Status changed from New to Closed

git commit c8a3ee0e3658c32402e6bd505596d4fa45bfe17c

commit  c8a3ee0e3658c32402e6bd505596d4fa45bfe17c
Author: Bryan Cantrill <bryan@joyent.com>
Date:   2019-06-25T00:12:14.000Z

    11208 add mdb format character for jazzed-up binary output
    11206 mdb output autowrapping still subtly wrong
    11207 many 64-bit mdb format characters have insufficient width
    Reviewed by: Robert Mustacchi <rm@joyent.com>
    Reviewed by: Jordan Hendricks <jordan.hendricks@joyent.com>
    Reviewed by: Toomas Soome <tsoome@me.com>
    Reviewed by: Andy Fiddaman <andy@omniosce.org>
    Reviewed by: Garrett D'Amore <garrett@damore.org>
    Approved by: Dan McDonald <danmcd@joyent.com>

Also available in: Atom PDF