castor

Throw away static initialization of XMLClassDescriptorImpl

Details

  • Type: Improvement Improvement
  • Status: Closed Closed
  • Priority: Major Major
  • Resolution: Fixed
  • Affects Version/s: None
  • Fix Version/s: 1.3 rc1
  • Component/s: None
  • Labels:
    None
  • Environment:
    all
  • Number of attachments :
    1

Description

Currently XMLClassDecriptorImpl has a static attribute _naming which is statically initialized and used just once in the method toXMLName.

I want to remove this as e.g. JAXB works with a different naming then Castor itself - and also I personally dislike static stuff.

I already changed the code (I'll attach the patch soon) and did some tests and everything works fine.

Activity

Hide
Permalink
Joachim Grüneis added a comment -

I simply removed the static stuff - checked if all calls to toXMLName are done with already a generated xml name at hand and... everything looks good, all tests are successful!

Show
Joachim Grüneis added a comment - I simply removed the static stuff - checked if all calls to toXMLName are done with already a generated xml name at hand and... everything looks good, all tests are successful!

People

  • Assignee:
    Joachim Grüneis
    Reporter:
    Joachim Grüneis
Vote (0)
Watch (0)

Dates

  • Created:
    Updated:
    Resolved: