[perl #25232] [PATCH] PIO_read bugs on reading larger chunks

2004-01-22 Thread Tamilmani, Arvindh Rajesh \(Cognizant\)
# New Ticket Created by  Tamilmani, Arvindh Rajesh (Cognizant) 
# Please include the string:  [perl #25232]
# in the subject line of all future correspondence about this issue. 
# URL: http://rt.perl.org/rt3/Ticket/Display.html?id=25232 


The attached patch 
a) fixes bugs in reading a larger chunk when the read buffer is not empty 
(io/io_buf.c).
b) corresponding test cases (t/src/io.t).
c) minor comment fix (io/io.c).

arvindh


bufread.patch
Description: Binary data
This e-mail and any files transmitted with it are for the sole use of the intended 
recipient(s) and may contain confidential and privileged information.
If you are not the intended recipient, please contact the sender by reply e-mail and 
destroy all copies of the original message. 
Any unauthorised review, use, disclosure, dissemination, forwarding, printing or 
copying of this email or any action taken in reliance on this e-mail is strictly 
prohibited and may be unlawful.

Visit us at http://www.cognizant.com


RE: [perl #25129] IO Buffer Test

2004-01-22 Thread Tamilmani, Arvindh Rajesh (Cognizant)
Even if that works, PASM tests shouldn't be in t/src but in t/pmc/io.t.

Sorry, I didn't know that.
The attached patch is for t/pmc/io.t

leo

Thanks,
Arvindh


biotests2.patch
Description: biotests2.patch
This e-mail and any files transmitted with it are for the sole use of the intended 
recipient(s) and may contain confidential and privileged information.
If you are not the intended recipient, please contact the sender by reply e-mail and 
destroy all copies of the original message. 
Any unauthorised review, use, disclosure, dissemination, forwarding, printing or 
copying of this email or any action taken in reliance on this e-mail is strictly 
prohibited and may be unlawful.

Visit us at http://www.cognizant.com


RE: [perl #25129] IO Buffer Test

2004-01-21 Thread Tamilmani, Arvindh Rajesh (Cognizant)
This is a test for the buffer bug that
was discussed on p6i a few days ago.

It does not remove the test file it makes
because I do not know a good way to do this.

You may use teardown() defined in t/src/io.t

The attached patch contains stripped down versions of your program
that I used to simulate the buffer IO bugs (perl #24941).

Test cases for:
a) PIO_flush on buffer full
b) turn off buffering

Leo, even with old versions of the repository, I couldn't simulate the 'filling the 
disk on flush' bug.
My box is i386-linux with gcc.

/Stefan

Thanks,
Arvindh


biotests.patch
Description: biotests.patch
This e-mail and any files transmitted with it are for the sole use of the intended 
recipient(s) and may contain confidential and privileged information.
If you are not the intended recipient, please contact the sender by reply e-mail and 
destroy all copies of the original message. 
Any unauthorised review, use, disclosure, dissemination, forwarding, printing or 
copying of this email or any action taken in reliance on this e-mail is strictly 
prohibited and may be unlawful.

Visit us at http://www.cognizant.com


[PATCH] RE: More Buffer IO Bugs (was: Strangeness when printing to file)

2004-01-18 Thread Tamilmani, Arvindh Rajesh (Cognizant)
 When running this I would expect the 2 files to
 look the same, but they don't.

Additionally to the reported bug these seem to be broken:

1) Turning off buffering:

The attached patch fixes the above two bugs:
a) when the buffer is full, it is not completely flushed; fixed.
b) setbuf sets b-size to PIO_GRAIN (and wrongly sets io-flags) when
   passed 0.  The patch sets b-size to 0 - hope it is the expected
   behavior.

Tests, tests, tests, and fixes very welcome,
leo

arvindh


biofixes.patch
Description: biofixes.patch
This e-mail and any files transmitted with it are for the sole use of the intended 
recipient(s) and may contain confidential and privileged information.
If you are not the intended recipient, please contact the sender by reply e-mail and 
destroy all copies of the original message. 
Any unauthorised review, use, disclosure, dissemination, forwarding, printing or 
copying of this email or any action taken in reliance on this e-mail is strictly 
prohibited and may be unlawful.

Visit us at http://www.cognizant.com


RE: CVS troubles

2004-01-18 Thread Tamilmani, Arvindh Rajesh (Cognizant)
 do
  $ cvs update -A classes/timer.pmc
 to remove the sticky tag and try committing again.

Ah, yes thanks, that worked.

If you wish to get a particular revision/branch of a file,
but don't want the sticky tag to be set, you may use this command:

$ cvs update -r tag -p file file

This way, Working revision and Sticky Tag will remain unchanged.

arvindh
This e-mail and any files transmitted with it are for the sole use of the intended 
recipient(s) and may contain confidential and privileged information.
If you are not the intended recipient, please contact the sender by reply e-mail and 
destroy all copies of the original message. 
Any unauthorised review, use, disclosure, dissemination, forwarding, printing or 
copying of this email or any action taken in reliance on this e-mail is strictly 
prohibited and may be unlawful.

Visit us at http://www.cognizant.com


[perl #24941] [PATCH] RE: More Buffer IO Bugs (was: Strangeness when printing to file)

2004-01-18 Thread Tamilmani, Arvindh Rajesh \(Cognizant\)
# New Ticket Created by  Tamilmani, Arvindh Rajesh (Cognizant) 
# Please include the string:  [perl #24941]
# in the subject line of all future correspondence about this issue. 
# URL: http://rt.perl.org/rt3/Ticket/Display.html?id=24941 


 When running this I would expect the 2 files to
 look the same, but they don't.

Additionally to the reported bug these seem to be broken:

1) Turning off buffering:

The attached patch fixes the above two bugs:
a) when the buffer is full, it is not completely flushed; fixed.
b) setbuf sets b-size to PIO_GRAIN (and wrongly sets io-flags) when
   passed 0.  The patch sets b-size to 0 - hope it is the expected
   behavior.

Tests, tests, tests, and fixes very welcome,
leo

arvindh


biofixes.patch
Description: Binary data
This e-mail and any files transmitted with it are for the sole use of the intended 
recipient(s) and may contain confidential and privileged information.
If you are not the intended recipient, please contact the sender by reply e-mail and 
destroy all copies of the original message. 
Any unauthorised review, use, disclosure, dissemination, forwarding, printing or 
copying of this email or any action taken in reliance on this e-mail is strictly 
prohibited and may be unlawful.

Visit us at http://www.cognizant.com


RE: CVS troubles (was: IMCC v1 call for bug list and feature freeze)

2004-01-17 Thread Tamilmani, Arvindh Rajesh (Cognizant)
cvs server: sticky tag `HEAD' for file `classes/timer.pmc' is 
not a branch

do
$ cvs update -A classes/timer.pmc
to remove the sticky tag and try committing again.

The sticky tag gets set when cvs checkout/update is done with 
the -r option.

leo

HTH,
arvindh.
This e-mail and any files transmitted with it are for the sole use of the intended 
recipient(s) and may contain confidential and privileged information.
If you are not the intended recipient, please contact the sender by reply e-mail and 
destroy all copies of the original message. 
Any unauthorised review, use, disclosure, dissemination, forwarding, printing or 
copying of this email or any action taken in reliance on this e-mail is strictly 
prohibited and may be unlawful.

Visit us at http://www.cognizant.com