Skip to content

Remove generics #1

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
wants to merge 1 commit into from
Closed

Conversation

hgschmie
Copy link

Remove the generic from Appender and subsequently change all the places that
pull this generic through the code base. This is mostly removal of places that try
to be generic but are not.

This also removes the need for a lot of casts, annotations and <?> generics.

Also fix a large number of raw class usages where generic classes such
as Map or Class are not used with generics. Explicitly do not touch
the APIs we implement (log4j 1.2 etc.).

…laces that

pull this generic through the code base. This is mostly removal of places that try
to be generic but are not.

This also removes the need for a lot of casts, annotations and <?> generics.

Also fix a large number of raw class usages where generic classes such
as Map or Class are not used with generics. Explicitly do not touch
the APIs we implement (log4j 1.2 etc.).
@jvz
Copy link
Member

jvz commented Jan 6, 2014

You wanna submit this as a patch on the JIRA tracker?

@Humbedooh
Copy link
Member

one more test... :C

@Humbedooh
Copy link
Member

looks like it works now :)

@asfgit asfgit closed this in 6e5c5c6 Apr 27, 2016
rgoers pushed a commit that referenced this pull request Jul 29, 2020
* Initial implementation of Redis appender.

* Add unit test support for Redis Appender.

* Clean up implementation.

* Add site documentation and clean up imports.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants