Mailing-List: contact derby-dev-help@db.apache.org; run by ezmlm
Precedence: bulk
Reply-To: <derby-dev@db.apache.org>
Message-ID: <19734128.1160679696312.JavaMail.jira@brutus>
Date: Thu, 12 Oct 2006 12:01:36 -0700 (PDT)
From: "Bryan Pendleton (JIRA)" <derby-dev@db.apache.org>
To: derby-dev@db.apache.org
Subject: [jira] Commented: (DERBY-1956) Remove stale code from the statement
 classes in the client driver
In-Reply-To: <28213461.1160638176348.JavaMail.jira@brutus>
MIME-Version: 1.0
Content-Type: text/plain; charset=utf-8
Content-Transfer-Encoding: 7bit

    [ http://issues.apache.org/jira/browse/DERBY-1956?page=comments#action_12441836 ] 
            
Bryan Pendleton commented on DERBY-1956:
----------------------------------------

Thanks Knut Anders and Dan for the clarification. It makes sense to me, and I
think that removing the dead code sounds good.

> Remove stale code from the statement classes in the client driver
> -----------------------------------------------------------------
>
>                 Key: DERBY-1956
>                 URL: http://issues.apache.org/jira/browse/DERBY-1956
>             Project: Derby
>          Issue Type: Improvement
>          Components: Network Client
>    Affects Versions: 10.3.0.0
>            Reporter: Knut Anders Hatlen
>         Assigned To: Knut Anders Hatlen
>            Priority: Minor
>             Fix For: 10.3.0.0
>
>         Attachments: 1956.diff, 1956.stat
>
>
> Statement and ParameterMetaData contain flags called escapedProcedureCallWithResult_. These flags might have had a function before, but now they are always false. They are used in many if-statements and make the code harder to read. Removing the flags and the code that is only executed when the flags are true, would simplify the code and make it more maintainable.

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: http://issues.apache.org/jira/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira