Deepa Remesh <[EMAIL PROTECTED]> writes:

> On 2/24/06, Knut Anders Hatlen <[EMAIL PROTECTED]> wrote:
>> You might also consider getting rid of pkgcnstkn, pkgid and pkgsn
>> since they are stored in pkgnamcsn anyway. If you remove them, we only
>> have two variables that aren't reset like the others.
>
> It will be good to get rid of pkgcnstkn, pkgid and pkgsn (in next
> patch). There are 2 more variables which get assigned only in
> setPkgnamcsn() - cursorName and isolationLevel. So we have this
> comment - " ALL variables (except 'database' and variables which are
> set in 'setPkgnamcsn' based on the 'pkgnamcsn' value)" ?

The comments you have added to DRDAResultSet and DRDAStatement are
very good. Thanks for doing the cleanup!

-- 
Knut Anders

Reply via email to