Qpsmtpd.t

furthering Issue #67 
The above code strikes me as odd.  Although I see you haven't changed functionality, I wonder if the removal of the comment is appropriate:

 # note this is wrong as $tran is always true in the
 # current code...

It seems to me that storing plugin return values in a place that changes depending on the state of the transaction is a bit strange.  I suspect it works because:

1) Nobody actually checks these return values, so it doesn't matter where they go
2) It's broken as the comment suggests, so all the values do actually go to the same place.

If it's #1, we ought to remove this code.
If it's #2, perhaps the comment should stay until the whole situation is worked out.
If instead, something here actually did change functionality such that the comment is no longer valid, are we sure that the code actually works as intended?

Anyway, for this PR, my concern is:  1) Do we know that comment really ought to be removed?