66617 2011-08-19 18:24:06 -0700 Only #include <signal.h> and require SA_RESTART when building with JSC_MULTIPLE_THREADS 2011-08-19 19:59:26 -0700 1 1 1 Unclassified WebKit JavaScriptCore 528+ (Nightly build) Unspecified Unspecified RESOLVED FIXED P2 Normal --- 1 dbates dbates webkit.review.bot oldest_to_newest 454262 0 dbates 2011-08-19 18:24:06 -0700 SA_RESTART is only applicable when building with JSC_MULTIPLE_THREADS enabled. On QNX we use Pthreads and don't enable JSC_MULTIPLE_THREADS. Currently, we require a platform to have SA_RESTART support even when JSC_MULTIPLE_THREADS is disabled. But SA_RESTART support is only applicable when JSC_MULTIPLE_THREADS is enabled. Hence, we should only require SA_RESTART support when JSC_MULTIPLE_THREADS is enabled. 454267 1 104607 dbates 2011-08-19 18:38:51 -0700 Created attachment 104607 Patch 454286 2 104607 webkit.review.bot 2011-08-19 19:59:22 -0700 Comment on attachment 104607 Patch Clearing flags on attachment: 104607 Committed r93467: <http://trac.webkit.org/changeset/93467> 454287 3 webkit.review.bot 2011-08-19 19:59:26 -0700 All reviewed patches have been landed. Closing bug. 104607 2011-08-19 18:38:51 -0700 2011-08-19 19:59:22 -0700 Patch bug-66617-20110819183605.patch text/plain 1522 dbates SW5kZXg6IFNvdXJjZS9KYXZhU2NyaXB0Q29yZS9DaGFuZ2VMb2cKPT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PQotLS0gU291 cmNlL0phdmFTY3JpcHRDb3JlL0NoYW5nZUxvZwkocmV2aXNpb24gOTM0NjIpCisrKyBTb3VyY2Uv SmF2YVNjcmlwdENvcmUvQ2hhbmdlTG9nCSh3b3JraW5nIGNvcHkpCkBAIC0xLDMgKzEsMTcgQEAK KzIwMTEtMDgtMTkgIERhbmllbCBCYXRlcyAgPGRiYXRlc0B3ZWJraXQub3JnPgorCisgICAgICAg IE9ubHkgI2luY2x1ZGUgPHNpZ25hbC5oPiBhbmQgcmVxdWlyZSBTQV9SRVNUQVJUIHdoZW4gYnVp bGRpbmcgd2l0aCBKU0NfTVVMVElQTEVfVEhSRUFEUworICAgICAgICBodHRwczovL2J1Z3Mud2Vi a2l0Lm9yZy9zaG93X2J1Zy5jZ2k/aWQ9NjY2MTcKKworICAgICAgICBCb3RoIDxzaWduYWwuaD4g YW5kIFNBX1JFU1RBUlQgdXNhZ2UgYXJlIGd1YXJkZWQgYmVoaW5kIEVOQUJMRShKU0NfTVVMVElQ TEVfVEhSRUFEUykuCisgICAgICAgIEJ1dCB3ZSBjYXVzZSBhIGNvbXBpbGUgZXJyb3IgaWYgdGhl IHBsYXRmb3JtIGRvZXNuJ3Qgc3VwcG9ydCBTQV9SRVNUQVJUIHJlZ2FyZGxlc3Mgb2YKKyAgICAg ICAgd2hldGhlciBKU0NfTVVMVElQTEVfVEhSRUFEUyBpcyBlbmFibGVkIGZvciB0aGUgcG9ydC4g SW5zdGVhZCwgd2Ugc2hvdWxkbid0IHJlcXVpcmUKKyAgICAgICAgU0FfUkVTVEFSVCBzdXBwb3J0 IHVubGVzcyB3ZSBhcmUgYnVpbGRpbmcgd2l0aCBKU0NfTVVMVElQTEVfVEhSRUFEUyBlbmFibGVk LgorCisgICAgICAgIFJldmlld2VkIGJ5IE5PQk9EWSAoT09QUyEpLgorCisgICAgICAgICogaGVh cC9NYWNoaW5lU3RhY2tNYXJrZXIuY3BwOgorCiAyMDExLTA4LTE5ICBEYW5pZWwgQmF0ZXMgIDxk YmF0ZXNAd2Via2l0Lm9yZz4KIAogICAgICAgICBJbXBsZW1lbnQgRXhlY3V0YWJsZUFsbG9jYXRv cjo6Y2FjaGVGbHVzaCgpIGZvciBRTlgKSW5kZXg6IFNvdXJjZS9KYXZhU2NyaXB0Q29yZS9oZWFw L01hY2hpbmVTdGFja01hcmtlci5jcHAKPT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PQotLS0gU291cmNlL0phdmFTY3JpcHRD b3JlL2hlYXAvTWFjaGluZVN0YWNrTWFya2VyLmNwcAkocmV2aXNpb24gOTM0NTYpCisrKyBTb3Vy Y2UvSmF2YVNjcmlwdENvcmUvaGVhcC9NYWNoaW5lU3RhY2tNYXJrZXIuY3BwCSh3b3JraW5nIGNv cHkpCkBAIC03Niw3ICs3Niw3IEBACiAjaW5jbHVkZSA8ZXJybm8uaD4KICNlbmRpZgogCi0jaWYg VVNFKFBUSFJFQURTKSAmJiAhT1MoV0lORE9XUykgJiYgIU9TKERBUldJTikKKyNpZiBFTkFCTEUo SlNDX01VTFRJUExFX1RIUkVBRFMpICYmIFVTRShQVEhSRUFEUykgJiYgIU9TKFdJTkRPV1MpICYm ICFPUyhEQVJXSU4pCiAjaW5jbHVkZSA8c2lnbmFsLmg+CiAjaWZuZGVmIFNBX1JFU1RBUlQKICNl cnJvciBNYWNoaW5lVGhyZWFkcyByZXF1aXJlcyBTQV9SRVNUQVJUCg==