kiall / android-tvheadend

Android + TV Headend + TV Input Framework
Apache License 2.0
191 stars 61 forks source link

rename Factory to HtspFactory to remove confusion with the parent class #176

Closed girlpunk closed 6 years ago

kiall commented 7 years ago

I'm not sure this really makes it any clearer, we have a HtspDataSource.Factory and a HtspExtractor.Factory.

Since these are inner classes, they are always referenced in the context of their parent class - making the other choice (HtspDataSource.HtspDataSourceFactory and HtspExtractor.HtspExtractorFactory) unnecessarily verbose)..

kiall commented 7 years ago

It also looks like PR #192 will change these classes around s bit, as introduce another new DataSource.

girlpunk commented 6 years ago

Closing