Skip Menu |

This queue is for tickets about the DBI CPAN distribution.

Report information
The Basics
Id: 130194
Status: new
Priority: 0/
Queue: DBI

Owner: Nobody in particular
Requestors: ppisar [...]

Bug Information
Severity: (no value)
Broken in: (no value)
Fixed in: (no value)

Subject: DBI::preparse() always copies ps_accept to ps_return
Download (untitled) / with headers
text/plain 562b
preparse() contins this code: if (!(ps_return | DBIpp_ph_XX)) { /* no return ph type specified */ ps_return |= ps_accept | DBIpp_ph_XX; /* so copy from ps_accept */ } The condition looks weird as DBIpp_ph_XX constant is 0x000F00, thus the condition is always true. As a result ps_return always gets the bits copied and 0x000F00 set. If that were a typo and one used if (!(ps_return & DBIpp_ph_XX)), t/60preparse.t test fails. If I simplify the code to: ps_return |= ps_accept; /* so copy from ps_accept */ The test still passes.

This service is sponsored and maintained by Best Practical Solutions and runs on infrastructure.

Please report any issues with to