MzHeader and PeHeader classes in PELIB were replaced by ImageLoader class

[ladislav-zezula]

No description provided.

[PeterMatula]

Thanks for this, the whole PE parsing is much better now.

In comments, there are few things I noticed. Not everything have to be fixed, some are just questions, or things to think about.

Some general remarks for newly written code (or major refactorization):

• Use strongly typed enums as default. Use old enums only if there is some specific reason why not to use strong enums.
• Initialize all class/struct members right at their declaration (e.g. type* whatever = nullptr;).
• Initialize all local variables right at their declaration, even if their are later defined before used - the code might change later, and there might be a problem (e.g. type* whatever = nullptr;)
• Initialize all ... = only those types, that do not have default constructors, so don't use something like this std::string str = "";
• Long lines (and there are many in PeLib/Fileformat/Fileinfo/...) hurt my head. No need to refactor all the old code, by in the new code, try to keep lines under 80 chars. Set a nice red line in your IDE :-D Break all the lines that are longer.
  This is much nicer:

void DebugDirectory::read(
    ImageLoader & imageLoader, 
    std::vector<PELIB_IMG_DEBUG_DIRECTORY> & debugInfo, 
    std::uint32_t rva, 
    std::uint32_t size
)

than this:

void DebugDirectory::read(ImageLoader & imageLoader, std::vector<PELIB_IMG_DEBUG_DIRECTORY> & debugInfo, std::uint32_t rva, std::uint32_t size)

[PeterMatula]

IDE with line break border. It is much nicer to have 2 or even 3 tabs side-by-side if the lines are short.