[webkit-reviews] review canceled: [Bug 181916] [GTK] Reenable -fvisibility=hidden : [Attachment 440201] Patch

[bugzilla-daemon at webkit.org]

Michael Catanzaro <mcatanzaro at gnome.org> has canceled  review:
Bug 181916: [GTK] Reenable -fvisibility=hidden
https://bugs.webkit.org/show_bug.cgi?id=181916

Attachment 440201: Patch

https://bugs.webkit.org/attachment.cgi?id=440201&action=review




--- Comment #70 from Michael Catanzaro <mcatanzaro at gnome.org> ---
Comment on attachment 440201
  --> https://bugs.webkit.org/attachment.cgi?id=440201
Patch

Actually I'm removing my r+ because I think this might break the
check-for-invalid-symbols-in-version-script safety check that you added in bug
#179914. If -fvisibility=hidden is used, then symbols that don't use BEXPORT or
WTF_EXPORT_PRIVATE could wind up duplicated, right? That would reintroduce bug
#179914?