:-1: mmm but the idea of the Loader settings is for future proofing. You could add a new var in LoaderSettings and not have to change the public signature of this class.

I'm not sure if you're simplification is just going to more of a maintenance headache.

I would have expected you to create another object `NetworkSettings` that encapsulates these 5 fields, you can still have this constructor but it should be private and `UrlNetworkManager(NetworkSettings` should be available.

If you want to discuss the ImageLoader I think we need a big piece of paper, draw all the classes that are publicly exposed on it and we will discuss what is needed and what isn't. Think of the children!