PE: TLS parser rework

[kkent030315]

• PE: TLS parser generics and bitness-related rework #433

Changelists:

• Removed generics from ImageTlsDirectory::parse* and TlsData::parse*, instead added new argument is_64 as proposed in the issue.
• Added slice bound check in TlsData::raw_data parser to compliant with panic-free design in goblin.

I personally prefer not making ImageTlsDirectory32―as it's more comfortable than the way duplicating the same kind of structs and merging to 64-bit as proposed in the issue.

This change should be marked as breaking-change and I expect this to be rolled up in 0.10.

[m4b]

this looks good, but I don't see where is_64 gets computed? i only see it passed to parse_with_opts ?

[kkent030315]

@m4b is_64 is already computed in PE::parse_with_opts and passed straight to the TlsData::parse_with_opts.

[m4b]

thank you for making these changes, the public api is more idiomatic again

[m4b]

NB: breaking change