Fix minor warnings

[Captain1653]

Fix minor warnings:
remove unnecessary generic type for collections,
replace Charset.forName on StandartCharsets

[filiphr]

Thanks for the PR @Captain1653.

I have some comments about it though.

Regarding the changes for changing the for loops with Collectors.toList(). There is one small performance problem about it. The performance is linked with the size of the list.

For example

List<String> strings = createStrings();
List<Integer> result = new ArrayList<>( strings.size() );

for (String string: strings) {
    result.add( Integer.parse( string ) );
}

vs

List<String> strings = createStrings();
List<Integer> result = strings.stream().map( Integer::parse ).collect( Collectors.toList() );

Both are semantically the same. However, in the first approach the result list is created with the expected size and in the second one it isn't. This means that when creating the result there will be a lot of expansions of the underlying array in ArrayList. In theory this could be done with Collectors.tolCollection( () -> new ArrayList( strings.size() ) ). However, I am not sure whether that is better or not.

It is similar for doing the remove in the getThrownTypes()

[Captain1653]

@filiphr
I think you are right about size of ArrayList and expansions. I can replace it with custom supplier or revert this changes. What is better?

P.S. From javadoc ArrayList java 8:

An empty ArrayList with elementData == DEFAULTCAPACITY_EMPTY_ELEMENTDATA will be expanded to DEFAULT_CAPACITY when the first element is added.

[filiphr]

I think that for the time being it is better to leave the for loops as is.

Not following what you mean about the javadoc.

[filiphr]

Thanks for applying the changes @Captain1653. Will merge it once the build is done