I've sent this in July, but it's not appeared in the archive.
No-where does it say the list is subscribers-only, so dunno;
re-sending.

Hi!

I've noticed this recently when digging through old trees;
this has been perpetrated since 4.4BSD[-Alpha],
which replaced V7's (32V's) dd: the "ibm" and "oldibm" tables
are identical, because POSIX just standardised the table from V7.
No-body, including the original authors, seeems to have noticed this.

This patch merges them and notes this fact in the manual
(and also fixes the odd, inconsistent, spelling of "pre-4.3BSD-Reno").

Best,
наб

---
diff --git a/dd.orig/args.c b/dd/args.c
index 748f52c..723dc72 100644
--- a/dd.orig/args.c
+++ b/dd/args.c
@@ -354,13 +354,13 @@ static const struct conv {
        { "ascii",      C_ASCII,        C_EBCDIC,       e2a_POSIX },
        { "block",      C_BLOCK,        C_UNBLOCK,      NULL },
        { "ebcdic",     C_EBCDIC,       C_ASCII,        a2e_POSIX },
-       { "ibm",        C_EBCDIC,       C_ASCII,        a2ibm_POSIX },
+       { "ibm",        C_EBCDIC,       C_ASCII,        a2ibm },
        { "lcase",      C_LCASE,        C_UCASE,        NULL },
        { "noerror",    C_NOERROR,      0,              NULL },
        { "notrunc",    C_NOTRUNC,      0,              NULL },
        { "oldascii",   C_ASCII,        C_EBCDIC,       e2a_32V },
        { "oldebcdic",  C_EBCDIC,       C_ASCII,        a2e_32V },
-       { "oldibm",     C_EBCDIC,       C_ASCII,        a2ibm_32V },
+       { "oldibm",     C_EBCDIC,       C_ASCII,        a2ibm },
        { "osync",      C_OSYNC,        C_BS,           NULL },
        { "sparse",     C_SPARSE,       0,              NULL },
        { "swab",       C_SWAB,         0,              NULL },
diff --git a/dd.orig/conv_tab.c b/dd/conv_tab.c
index 89d32da..dd87e15 100644
--- a/dd.orig/conv_tab.c
+++ b/dd/conv_tab.c
@@ -142,42 +142,6 @@ const u_char a2e_32V[] = {
        0356, 0357, 0372, 0373, 0374, 0375, 0376, 0377,         /* 0370 */
 };
 
-/* ASCII to IBM EBCDIC -- 32V compatible. */
-const u_char a2ibm_32V[] = {
-       0000, 0001, 0002, 0003, 0067, 0055, 0056, 0057,         /* 0000 */
-       0026, 0005, 0045, 0013, 0014, 0015, 0016, 0017,         /* 0010 */
-       0020, 0021, 0022, 0023, 0074, 0075, 0062, 0046,         /* 0020 */
-       0030, 0031, 0077, 0047, 0034, 0035, 0036, 0037,         /* 0030 */
-       0100, 0132, 0177, 0173, 0133, 0154, 0120, 0175,         /* 0040 */
-       0115, 0135, 0134, 0116, 0153, 0140, 0113, 0141,         /* 0050 */
-       0360, 0361, 0362, 0363, 0364, 0365, 0366, 0367,         /* 0060 */
-       0370, 0371, 0172, 0136, 0114, 0176, 0156, 0157,         /* 0070 */
-       0174, 0301, 0302, 0303, 0304, 0305, 0306, 0307,         /* 0100 */
-       0310, 0311, 0321, 0322, 0323, 0324, 0325, 0326,         /* 0110 */
-       0327, 0330, 0331, 0342, 0343, 0344, 0345, 0346,         /* 0120 */
-       0347, 0350, 0351, 0255, 0340, 0275, 0137, 0155,         /* 0130 */
-       0171, 0201, 0202, 0203, 0204, 0205, 0206, 0207,         /* 0140 */
-       0210, 0211, 0221, 0222, 0223, 0224, 0225, 0226,         /* 0150 */
-       0227, 0230, 0231, 0242, 0243, 0244, 0245, 0246,         /* 0160 */
-       0247, 0250, 0251, 0300, 0117, 0320, 0241, 0007,         /* 0170 */
-       0040, 0041, 0042, 0043, 0044, 0025, 0006, 0027,         /* 0200 */
-       0050, 0051, 0052, 0053, 0054, 0011, 0012, 0033,         /* 0210 */
-       0060, 0061, 0032, 0063, 0064, 0065, 0066, 0010,         /* 0220 */
-       0070, 0071, 0072, 0073, 0004, 0024, 0076, 0341,         /* 0230 */
-       0101, 0102, 0103, 0104, 0105, 0106, 0107, 0110,         /* 0240 */
-       0111, 0121, 0122, 0123, 0124, 0125, 0126, 0127,         /* 0250 */
-       0130, 0131, 0142, 0143, 0144, 0145, 0146, 0147,         /* 0260 */
-       0150, 0151, 0160, 0161, 0162, 0163, 0164, 0165,         /* 0270 */
-       0166, 0167, 0170, 0200, 0212, 0213, 0214, 0215,         /* 0300 */
-       0216, 0217, 0220, 0232, 0233, 0234, 0235, 0236,         /* 0310 */
-       0237, 0240, 0252, 0253, 0254, 0255, 0256, 0257,         /* 0320 */
-       0260, 0261, 0262, 0263, 0264, 0265, 0266, 0267,         /* 0330 */
-       0270, 0271, 0272, 0273, 0274, 0275, 0276, 0277,         /* 0340 */
-       0312, 0313, 0314, 0315, 0316, 0317, 0332, 0333,         /* 0350 */
-       0334, 0335, 0336, 0337, 0352, 0353, 0354, 0355,         /* 0360 */
-       0356, 0357, 0372, 0373, 0374, 0375, 0376, 0377,         /* 0370 */
-};
-
 /* EBCDIC to ASCII -- POSIX and System V compatible. */
 const u_char e2a_POSIX[] = {
        0000, 0001, 0002, 0003, 0234, 0011, 0206, 0177,         /* 0000 */
@@ -250,8 +214,8 @@ const u_char a2e_POSIX[] = {
        0356, 0357, 0372, 0373, 0374, 0375, 0376, 0377,         /* 0370 */
 };
 
-/* ASCII to IBM EBCDIC -- POSIX and System V compatible. */
-const u_char a2ibm_POSIX[] = {
+/* ASCII to IBM EBCDIC -- 32V, POSIX and System V compatible. */
+const u_char a2ibm[] = {
        0000, 0001, 0002, 0003, 0067, 0055, 0056, 0057,         /* 0000 */
        0026, 0005, 0045, 0013, 0014, 0015, 0016, 0017,         /* 0010 */
        0020, 0021, 0022, 0023, 0074, 0075, 0062, 0046,         /* 0020 */
diff --git a/dd.orig/dd.1 b/dd/dd.1
index 8ec7ab6..f5cb8ed 100644
--- a/dd.orig/dd.1
+++ b/dd/dd.1
@@ -293,8 +293,8 @@ The value
 .Cm oldascii
 specifies the one used in historic
 .Tn AT&T
-and pre-
-.Bx 4.3 Reno
+and
+.Pf pre- Bx 4.3 Reno
 systems.
 .It Cm block
 Treats the input as a sequence of newline or end-of-file terminated variable
@@ -322,26 +322,26 @@ records are converted.
 if the operand
 .Cm cbs
 is also specified.)
-There are four conversion maps for
+There are three conversion maps for
 .Tn EBCDIC .
 The value
 .Cm ebcdic
 specifies the recommended one which is compatible with
 .At V .
-The value
+The values
 .Cm ibm
-is a slightly different mapping, which is compatible with the
-.At V
+and
+.Cm oldibm
+are equivalent and specify a slightly different mapping, compatible with the
+.At v7
 .Cm ibm
 value.
-The values
+The value
 .Cm oldebcdic
-and
-.Cm oldibm
-are maps used in historic
+is the map used in historic
 .Tn AT&T
-and pre
-.Bx 4.3 Reno
+and
+.Pf pre- Bx 4.3 Reno
 systems.
 .It Cm lcase
 Transform uppercase characters into lowercase characters.
diff --git a/dd.orig/extern.h b/dd/extern.h
index 989fc78..3e98b13 100644
--- a/dd.orig/extern.h
+++ b/dd/extern.h
@@ -81,7 +81,7 @@ extern uint64_t               progress;
 extern const u_char    *ctab;
 extern const u_char    a2e_32V[], a2e_POSIX[];
 extern const u_char    e2a_32V[], e2a_POSIX[];
-extern const u_char    a2ibm_32V[], a2ibm_POSIX[];
+extern const u_char    a2ibm[];
 extern u_char          casetab[];
 extern const char      *msgfmt;
 

Attachment: signature.asc
Description: PGP signature

Reply via email to