camel-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Babak Vahdat (Updated) (JIRA)" <>
Subject [jira] [Updated] (CAMEL-4899) Code Clean-Up (continued)
Date Sun, 15 Jan 2012 21:45:39 GMT


Babak Vahdat updated CAMEL-4899:

    Attachment: CAMEL-4899-remove-unused.patch

Attached the patch concerning "remove unused" with some extra sugar concerning:

- removal of the deprecated API usages (and corresponding SuppressWarning("deprecated")) in
bunch of the places like the API's:

new org.apache.camel.component.mock.MockEndpoint("mock:foo")
getIdentifier() method usage instead of getUUID() by javax.jcr.Node
getNodeByIdentifier() method usage instead of getNodeByUUID() by javax.jcr.Session

- still removal of some unnecessary casts
- still removal of some unused imports

Note: at the first glance the change made for


could seem that as if the removal of the fields

- noReplyLogger
- acceptorConfig
- sync

would be "too much" however looking deeply into it I found out that all those member variables
are *unused* or their usage-chain in the class itself were again *unused*.
> Code Clean-Up (continued)
> -------------------------
>                 Key: CAMEL-4899
>                 URL:
>             Project: Camel
>          Issue Type: Improvement
>    Affects Versions: 2.8.3
>            Reporter: Babak Vahdat
>            Assignee: Babak Vahdat
>             Fix For: 2.9.1, 2.10.0
>         Attachments: CAMEL-4899-remove-unused.patch
> As Hadrian has already proposed by CAMEL-4796 this ticket has been created as it's continuation.
The remaing tasks are therewith
> - Removal of unused private memebers: Types, Constructors, Fields and Methods where applicable
> - Remove the trailing whitespace on all lines, even on the empty ones

This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators:!default.jspa
For more information on JIRA, see:


View raw message