search

Pkmmte / PkRSS

A powerful RSS feed manager for Android
Apache License 2.0
100 stars 23 forks source link

Implement media content parsing for RSS feeds #19

Closed fusion44 closed 8 years ago

fusion44 commented 8 years ago

I've implemented media content parsing for RSS feeds.

More info here

Pkmmte commented 8 years ago

Thanks for the pull request!

Before I merge, I have one question: why did you decide to use a Vector instead of a synchronized List?

fusion44 commented 8 years ago

My pleasure!

I didn't use it because I didn't know about it. Now that I've read a bit more about Vectors in Java I'll change the code to a synchronized list. Vectors seem to be more or less deprecated. Sorry about that.

Pkmmte commented 8 years ago

I'd appreciate it if you could modify your pull request to use a regular ArrayList instead. I don't think synchronization is really necessary for this. :)

fusion44 commented 8 years ago

Sorry! It might take a few additional days to get this through. To much todo but not enough time :-) I'll do it next week at latest.

Pkmmte commented 8 years ago

I'll go ahead and merge it anyway. I'll take care of modifying it.

fusion44 commented 8 years ago

Thank you and sorry.