db-derby-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Knut Anders Hatlen (JIRA)" <j...@apache.org>
Subject [jira] Commented: (DERBY-4293) Mutable public static variables
Date Mon, 29 Jun 2009 12:57:47 GMT

    [ https://issues.apache.org/jira/browse/DERBY-4293?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12725156#action_12725156
] 

Knut Anders Hatlen commented on DERBY-4293:
-------------------------------------------

Thanks for the patch, Sebb! It looks fine to me. Committed revision 789313.

> Mutable public static variables
> -------------------------------
>
>                 Key: DERBY-4293
>                 URL: https://issues.apache.org/jira/browse/DERBY-4293
>             Project: Derby
>          Issue Type: Bug
>            Reporter: Sebb
>         Attachments: derbyfinal.patch
>
>
> There are quite a few instances of public static fields which appear to be intended to
be constant, but which are not final.
> The code would be safer if all the constant fields were made final.
> Patch to follow for some of the problem fields.
> The following also ought to be fixed:
> org.apache.derby.iapi.services.property.PropertyUtil.servicePropertyList (e.g. use an
accessor to lookup the values)
> org.apache.derby.iapi.types.JSQLType.primitiveNames (ditto)

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


Mime
View raw message