Re: Eclipse Compiler Warnings

Anthony Berglas <[email protected]>
Newsgroups gmane.comp.java.orm.simpleorm
Message-ID <[email protected]>
Hello Bruce,

1. Descritpive renamed.  Thanks.

2. The idea was that the field name need not always be the same as 
the sql column name.  But the code does not actually support the case 
where they are different.  I've removed the columnName field, but 
left getColumnName (with comment).  Might revisit this later.  Thanks.

3. We use the Java programming language as defined by Sun and the 
compilers.  Eclipse et al might wish to further restrict the language 
but that is generally their problem.  Things like accessing final 
statica via object variables is convenient in examples and not 
unreasonable given the code style, IMHO.  And we try to avoid 
cluttering the code with unnecessary casts.

I had a look but could not find any Serializables without version 
Ids.  (Not really needed for SimpleORM, but not a bad idea.)

If you have specific issues then please raise them.  I'm sure there are some!

Anthony

PS.  Never do aggressive reformatting of code you are not editing -- 
it kills diff.  Netbeans is awful in this regard.


At 01:23 AM 24/09/2008, Bruce Alspaugh wrote:

>Eclipse gives me lots of compiler warnings when I open the source
>code. They include unused imports, references to generic types that
>should be parameterized, static members being accessed in a non-static
>way, serializable classes that don't set version id's, unchecked
>casts, etc. Can these be cleaned up for the next release?
>
>Also, it looks like "SFieldMeta.isDescritpive()" is misspelled.
>
>What is the distinction between SFieldMeta.getFieldName() and
>SFieldScalar.getColumnName()?
>
>Bruce
>

Dr Anthony Berglas, [email protected]       Mobile: +61 4 4838 8874
Just because it is possible to push twigs along the ground with ones nose
does not necessarily mean that is the best way to collect firewood.

------------------------------------

Yahoo! Groups Links

<*> To visit your group on the web, go to:
    http://groups.yahoo.com/group/SimpleORM/

<*> Your email settings:
    Individual Email | Traditional

<*> To change settings online go to:
    http://groups.yahoo.com/group/SimpleORM/join
    (Yahoo! ID required)

<*> To change settings via email:
    mailto:[email protected] 
    mailto:[email protected]

<*> To unsubscribe from this group, send an email to:
    [email protected]

<*> Your use of Yahoo! Groups is subject to:
    http://docs.yahoo.com/info/terms/
9e7966.jpg (image/jpeg, 3.3 KB) - not displayed
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.