TProtocol.h: Be extra careful when including MSVC Windows-related headers

[emmenlau]

Windows-related headers can pollute the c++ namespace with macros min and max and other quirks that break standard c++ code quite easily.

• Did you create an Apache Jira ticket? (Request account here, not required for trivial changes)
• If a ticket exists: Does your pull request title follow the pattern "THRIFT-NNNN: describe my issue"?
• Did you squash your changes to a single commit? (not required, but preferred)
• Did you do your best to avoid breaking changes? If one was needed, did you label the Jira ticket with "Breaking-Change"?
• If your change does not involve any code, include [skip ci] anywhere in the commit message to free up build resources.

[emmenlau]

No build problems or errors are related to this change. Failing tests already failed before:

• https://ci.appveyor.com/project/ApacheSoftwareFoundation/thrift/builds/48258433/job/cyhd8al8agk2a91e
• https://ci.appveyor.com/project/ApacheSoftwareFoundation/thrift/builds/48258433/job/5cyffy00o3rafy6l