bugwdiff - Bugs: bug #25883, Better handling of non-seekable...

 
 

bug #25883: Better handling of non-seekable input

Submitted by:  Martin von Gagern <gagern>
Submitted on:  Mon 16 Mar 2009 11:12:37 AM UTC  
 
Category: NoneSeverity: 3 - Normal
Item Group: NoneStatus: Fixed
Privacy: PublicAssigned to: Martin von Gagern <gagern>
Originator Name: Open/Closed: Closed

Add a New Comment(Rich Markup)
   

You are not logged in

Please log in, so followups can be emailed to you.

 

Tue 15 Dec 2009 05:12:12 PM UTC, comment #1:

Fix comitted, will be included in next release.

Martin von Gagern <gagern>
Project AdministratorIn charge of this item.
Mon 16 Mar 2009 11:12:37 AM UTC, original submission:

I tend to use shell process substitutions a lot with diff. However, when I try to do the same with wdiff, I loose output without any indication that anything might have gone wrong. To test this, try the following:
wdiff <(echo here foo there) <(echo here bar there)

The primary cause of this is that this kind of input is not seekable. wdiff already contains some code to copy input to a temporary file, but it uses that method only for stdin.

The attached patch checks whether or not a stream is seekable, and copies it to a temporary buffer if not. With this patch in place, the above command produces the expected output. As an added benefit, in cases where stdin is seekable, unnecessary buffering is avoided, though this case is probably rare.

Martin von Gagern <gagern>
Project AdministratorIn charge of this item.

 

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

Attach File(s):
   
   
Comment:
   

Attached Files
file #17705:  NonSeekableInput.patch added by gagern (3KiB - application/octet-stream)

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -unavailable- added by gagern (Submitted the item)
  •  

    Do you think this task is very important?
    If so, you can click here to add your encouragement to it.
    This task has 0 encouragements so far.

    Only logged-in users can vote.

     

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

     

     

    Follow 4 latest changes.

    Date Changed By Updated Field Previous Value => Replaced By
    Tue 15 Dec 2009 05:12:12 PM UTCgagernStatusNone=>Fixed
      Assigned toNone=>gagern
      Open/ClosedOpen=>Closed
    Mon 16 Mar 2009 11:12:37 AM UTCgagernAttached File-=>Added NonSeekableInput.patch, #17705

    Back to the top


    Powered by Savane 3.1-cleanup1