Remove usage of reflection in timeago-parser module

[thigg]

We are trying to use the NewPipe extractor library for a native app for the mobile linux OS SailfishOS.
For this we intend to use graal's native-image.
When reflection is used, this needs an entry in a json file for each of the classes that may be used with reflection.

I created the put lines for the hashmap with a simple bashscript.
I think this should be totally equal (in terms of functionallity and maintainability) to the current solution without the drawback (for us) that it uses reflection.

• I carefully read the contribution guidelines and agree to them.
• I have tested the API against NewPipe.
• I agree to create a pull request for NewPipe as soon as possible to make it compatible with the changed API.

[Stypox]

Thank you! Yes it makes a lot of sense to remove reflection. All of the java files under the patterns/ folder were autogenerated by https://github.com/TeamNewPipe/NewPipeExtractor/blob/dev/timeago-parser/raw/java/GeneratePatternClasses.java, could you instead change that class to also autogenerate a java file which contains the hashmap, instead of manually hardcoding it?

[thigg]

Sure, sounds like no big deal.

[thigg]

@Stypox I updated the PR to generate the map with the generator.
While working at the generator I also moved it into its own gradle submodule, I hope this was okay?

[gechoto]

I guess this means the proguard rule mentioned in the readme can be removed:

NewPipeExtractor/README.md

Line 18 in 0b99100

-keep class org.schabi.newpipe.extractor.timeago.patterns.** { *; }

[thigg]

removed the line

[Stypox]

Thank you! I tested importing the project in Android Studio and running the generator and everything worked without issues. I solved the two comments myself with two commits.

[thigg]

Thank you. That was pleasant :)