bugPSPP - Bugs: bug #17100, DATA LIST doesn't properly handle...

 
 

bug #17100: DATA LIST doesn't properly handle empty trailing records

Submitter:  Ben Pfaff <blp>
Submitted:  Fri 14 Jul 2006 03:27:49 AM UTC
   
 
Category:  Other Severity:  3 - Ordinary
Priority:  5 - Normal Status:  Fixed
Assigned to:  blp Open/Closed:  Closed
Release:  Before 0.6.0 Effort:  0.00
* Mandatory Fields

Post a Comment

Add a New Comment Rich Markup
   

Discussion

Thu 26 Jul 2007 01:52:19 AM UTC, comment #5: 

I checked this in.

Ben Pfaff <blp>
Group administrator
Thu 26 Jul 2007 01:24:43 AM UTC, comment #4: 

Looks fine to me.

John Darrington <jmd>
Group administrator
Sun 22 Jul 2007 10:41:10 PM UTC, comment #3: 

The attached patch adds the tests called for by comment #1 and comment #2.  It also fixes the bug that the test for comment #2 revealed ;-)

(file #13431)

Ben Pfaff <blp>
Group administrator
Wed 19 Jul 2006 04:46:33 AM UTC, comment #2: 

Also, we need a test for multi-record fixed DATA LIST.  I don't think anything tests this right now.

Ben Pfaff <blp>
Group administrator
Mon 17 Jul 2006 04:22:08 AM UTC, comment #1: 

Actually, looking again, it handles them just fine.

What we really need is a test that makes sure that this case is handled.

Ben Pfaff <blp>
Group administrator
Fri 14 Jul 2006 03:27:49 AM UTC, original submission:  

It's possible to specify DATA LIST FIXED such that each case contains one or more trailing records whose data is unused.  It looks like the current implementation doesn't skip those records.  Should verify and fix if so.

Ben Pfaff <blp>
Group administrator

 

Attached Files

Attached Files

(Note: upload size limit is set to 4.0MiB, after insertion of the required escape characters.)

Attach Files:
   
   
Comment:
   

 

Dependencies

This item does not depend on any other items.

No items depend on this one.

 

Mail Notification Carbon-Copy List

Carbon-Copy List
  • -email is unavailable- added by jmd (Posted a comment)
  • -email is unavailable- added by blp (Updated the item)
  •  

    Votes

    There are 0 votes so far. Votes easily highlight which items people would like to see resolved in priority, independently of the priority of the item set by tracker managers.

    Only logged-in users can vote.

     

    Please enter the title of George Orwell's famous dystopian book (it's a date):

    History

    Follow 5 latest changes.

    Date Changed by Updated Field Previous Value => Replaced by
    2007-07-26 blp StatusReady for Test/Review Fixed
        Open/ClosedOpen Closed
    2007-07-22 blp Attached File- Added data-list-trailing-empty-records.patch, #13431
        StatusNone Ready for Test/Review
    2006-07-31 blp ReleaseNone Before 0.6.0

    Back to the top

    Powered by Savane 3.16-2753.
    Corresponding source code