Document Number: AUSTIN/117

Title: XSH/TC1/D1 Aardvark Change Request Report

Revision Date: 2002-06-26

Source: Andrew Josey, Chair

Action: for review

This report contains the draft dispositions of the aardvark comments
submitted against the XSH/TC1/D1  text.

Aardvark Summary Table
______________________
ERN 1 Accept as marked 
ERN 2 Accept 
ERN 3 Accept 
ERN 4 Accept 
ERN 5 Accept 
ERN 6 Accept 

 _____________________________________________________________________________
 OBJECTION                                        Enhancement Request Number 1
 Clive D.W. Feather                                BUG in XSHtc1-d1 (rdvk#  1)
                                               Tue, 28 May 2002 09:17:51 +0100
 _____________________________________________________________________________
 Accept_____    Accept as marked below_X___     Duplicate_____     Reject_____
 Rationale for rejected or partial changes:

 Accept the first part of the change,
 no need for part 2, is this already covered by suffix _t

 _____________________________________________________________________________
 Page: 1  Line: 27-28  Section: XSH/TC1/D1/3


 Problem:
 The reserved names given in TC1 don't match those in the C Standard.

 Action:
 Change the two lines to:

     INT[0-9A-Za-z_]*_MIN, INT[0-9A-Za-z_]*_MAX, INT[0-9A-Za-z_]*_C
     UINT[0-9A-Za-z_]*_MIN, UINT[0-9A-Za-z_]*_MAX, UINT[0-9A-Za-z_]*_C

 Additional comment:
 The C standard reserves the type names:
     int[0-9A-Za-z_]*_t, uint[0-9A-Za-z_]*_t
 for this header. I'm not sure if any corresponding change is needed.

 [Ed recommendation: Accept the change,
 not sure about part 2, is this already covered by suffix _t]


 _____________________________________________________________________________
 EDITORIAL                                        Enhancement Request Number 2
 Clive D.W. Feather                                BUG in XSHtc1-d1 (rdvk#  2)
                                               Tue, 28 May 2002 09:17:51 +0100
 _____________________________________________________________________________
 Accept_X___    Accept as marked below_____     Duplicate_____     Reject_____
 Rationale for rejected or partial changes:


 _____________________________________________________________________________
 Page: 1  Line: 67  Section: XSH/TC1/D1/6


 Problem:
 Bad grammar.

 Action:
 Change "elements of" to "an element of".
 Make the same change on line 387.

 [Ed recommendation:Accept]


 _____________________________________________________________________________
 EDITORIAL                                        Enhancement Request Number 3
 jonhitchcock@hotmail.com                                localtime  (rdvk#  6)
 {jjh73}                                  Fri, 7 Jun 2002 11:11:54 +0100 (BST)
 _____________________________________________________________________________
 Accept_X___    Accept as marked below_____     Duplicate_____     Reject_____
 Rationale for rejected or partial changes:


 _____________________________________________________________________________
 Page: 1  Line: 211,270  Section: gmtime,localtime


 Problem:

 The statements about gmtime() and localtime() returning a null pointer
 are said to be CX-shaded.  These statements are being added for
 alignment with the C standard, and the shading is unnecessary.

 Action:

 Remove "(Shaded and marked with the CX margin code)" from line 211 and
 from line 270.

 [Ed recommendation: Accept]


 _____________________________________________________________________________
 EDITORIAL                                        Enhancement Request Number 4
 jonhitchcock@hotmail.com               Bug in XSHtc1-d1 localtime  (rdvk#  5)
 {jjh74}                                  Fri, 7 Jun 2002 11:13:45 +0100 (BST)
 _____________________________________________________________________________
 Accept_X___    Accept as marked below_____     Duplicate_____     Reject_____
 Rationale for rejected or partial changes:


 _____________________________________________________________________________
 Page: 1  Line: 262  Section: localtime


 Problem:

 The section affected is wrongly given as "gmtime".

 Action:

 Change "gmtime" to "localtime".

 [Ed recommendation: Accept]


 _____________________________________________________________________________
 EDITORIAL                                        Enhancement Request Number 5
 jonhitchcock@hotmail.com                                  realpath (rdvk#  4)
 {jjh75}                                  Fri, 7 Jun 2002 11:15:41 +0100 (BST)
 _____________________________________________________________________________
 Accept_X___    Accept as marked below_____     Duplicate_____     Reject_____
 Rationale for rejected or partial changes:


 _____________________________________________________________________________
 Page: 1  Line: 347-392  Section: qsort,realpath


 Problem:

 Lines 347-365 (realpath) and 366-392 (qsort) are not in the same
 order as the sections they change.

 Action:

 Move lines 347-365 after line 392.

 [Ed recommendation: Accept]


 _____________________________________________________________________________
 OBJECTION                                        Enhancement Request Number 6
 Clive D.W. Feather                                BUG in XSHtc1-d1 (rdvk#  3)
                                               Tue, 28 May 2002 09:17:51 +0100
 _____________________________________________________________________________
 Accept_X___    Accept as marked below_____     Duplicate_____     Reject_____
 Rationale for rejected or partial changes:


 _____________________________________________________________________________
 Page: 1  Line: 368-392  Section: XSH/TC1/D1/35


 Problem:
 The wording used here applies to bcompare, not to qsort.

 Action:
 Lines 371-2: change "no match shall be found" to "no rearrangement shall
 take place".
 Delete lines 378-9.
 Replace lines 383-4 with: "That is, they shall define a total ordering on
 the array."
 Line 386: change "the second argument" to "each argument".
 Line 388: after for every call insert ", for each argument separately,".

 [Ed recommendation: Accept]