Remove include of obsolete header NSMapTablePriv.h.
(In reply to Daniel Bates from comment #0)
> Remove include of obsolete header NSMapTablePriv.h.
Actually, we need to continue including this header for a bit longer.
Created attachment 329655 [details]
Comment on attachment 329655 [details]
View in context: https://bugs.webkit.org/attachment.cgi?id=329655&action=review
> +#if USE(APPLE_INTERNAL_SDK) && PLATFORM(IOS) && __IPHONE_OS_VERSION_MAX_ALLOWED < 120000
This has the effect of using the below declarations in the __IPHONE_OS_VERSION_MAX_ALLOWED >= 120000 case. But I think we shouldn’t make those declarations in that case, because they are already bade in NSMapTable.h, which we import above. My understanding is that the entire SPI header is only needed for older SDK versions (which don’t have the declarations in their NSMapTable.h), so it would be nice to express that with guards around the entire thing.
Created attachment 329662 [details]
Created attachment 329663 [details]
Comment on attachment 329663 [details]
Clearing flags on attachment: 329663
Committed r226073: <https://trac.webkit.org/changeset/226073>
All reviewed patches have been landed. Closing bug.