Noel J. Bergman wrote: Just got back. Due to be sailing most of the weekend. Electric storms passing through, lightning conductor (mast) pointed heavenwards. Best get my comments in while I can.
<snipped> The code "looks" fine. I'ld be happier knowing that we've tested it with real world boundary cases. I guess that lacking unit tests, that's what release canidates are for. > > IMHO we should remove the > > AbstractRedirect#getExistingReturnPath(Mail mail) > > method as it becomes misleading and is no longer used. > > I had thought about deprecating it, at the least. Removing it is fine > with me. Personally I think we should give fair warning prior to removing things, which means deprecating in this release and removing in a subsequent release. <snipped> -- Steve --------------------------------------------------------------------- To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
