10384 2006-08-14 00:52:49 -0700 Switch to DWARF for Release configuration 2006-08-15 09:02:42 -0700 1 1 1 Unclassified WebKit WebKit Misc. 420+ Mac OS X 10.4 RESOLVED FIXED P2 Normal --- 1 mrowe webkit-unassigned darin ddkilzer timothy oldest_to_newest 52512 0 mrowe 2006-08-14 00:52:49 -0700 WebKit was recently switched back to the STABS debug format after various issues were encountered while attempting to debug code that used the DWARF format. The only issues that I can recall hearing about were all related to debugging, so I would like to suggest that the Release configuration is moved back to DWARF so that those of us that primarily use that build style don't have to suffer through the horrendous link times that STABS brings. 52513 1 10018 mrowe 2006-08-14 00:57:33 -0700 Created attachment 10018 Patch 52514 2 ddkilzer 2006-08-14 02:29:35 -0700 In one of the sessions at WWDC, I asked the Xcode engineers specifically about the bugs found when trying to step through debug builds of WebKit with DWARF. They said that these bugs were filed too late in the release process for Xcode 2.4, but that they were being worked on. Thus I don't think DWARF is "ready for prime-time" yet with WebKit. :( But I take it that people have switched to Xcode 2.4 now? 52515 3 ddkilzer 2006-08-14 02:32:18 -0700 Oops, I just noticed that this was only for the Release configuration, not Debug. I suppose that wouldn't matter then? :) 52521 4 darin 2006-08-14 08:24:18 -0700 We do need to debug even the Release configuration, but this still might be a good change. 52648 5 timothy 2006-08-15 08:58:12 -0700 I think this will be fine for Release builds. We can try it for a while and see how it goes. 52649 6 timothy 2006-08-15 09:02:42 -0700 Landed in r15880. 10018 2006-08-14 00:57:33 -0700 2006-08-15 08:54:52 -0700 Patch webkit-bug-10384-v1.patch text/plain 4200 mrowe SW5kZXg6IEphdmFTY3JpcHRDb3JlL0NoYW5nZUxvZwo9PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBKYXZhU2NyaXB0 Q29yZS9DaGFuZ2VMb2cJKHJldmlzaW9uIDE1ODUzKQorKysgSmF2YVNjcmlwdENvcmUvQ2hhbmdl TG9nCSh3b3JraW5nIGNvcHkpCkBAIC0xLDMgKzEsMTAgQEAKKzIwMDYtMDgtMTQgIE1hcmsgUm93 ZSAgPG9wZW5kYXJ3aW4ub3JnQGJkYXNoLm5ldC5uej4KKworICAgICAgICBodHRwOi8vYnVnemls bGEub3BlbmRhcndpbi5vcmcvc2hvd19idWcuY2dpP2lkPTEwMzg0CisgICAgICAgIEJ1ZyAxMDM4 NDogU3dpdGNoIHRvIERXQVJGIGZvciBSZWxlYXNlIGNvbmZpZ3VyYXRpb24KKworICAgICAgICAq IEphdmFTY3JpcHRDb3JlLnhjb2RlcHJvai9wcm9qZWN0LnBieHByb2o6CisKIDIwMDYtMDgtMTMg IE1ha3MgT3Jsb3ZpY2ggIDxtYWtzaW1Aa2RlLm9yZz4KIAogICAgICAgICBSZXZpZXdlZCAoYW5k IHR3ZWFrZWQgYSBsaXR0bGUpIGJ5IE1hY2llai4KSW5kZXg6IEphdmFTY3JpcHRDb3JlL0phdmFT Y3JpcHRDb3JlLnhjb2RlcHJvai9wcm9qZWN0LnBieHByb2oKPT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PQotLS0gSmF2YVNj cmlwdENvcmUvSmF2YVNjcmlwdENvcmUueGNvZGVwcm9qL3Byb2plY3QucGJ4cHJvagkocmV2aXNp b24gMTU4NTEpCisrKyBKYXZhU2NyaXB0Q29yZS9KYXZhU2NyaXB0Q29yZS54Y29kZXByb2ovcHJv amVjdC5wYnhwcm9qCSh3b3JraW5nIGNvcHkpCkBAIC0xNzg2LDYgKzE3ODYsNyBAQAogCQkxNDlD Mjc3MzA4OTAyQUZFMDA4QTlFRkMgLyogUmVsZWFzZSAqLyA9IHsKIAkJCWlzYSA9IFhDQnVpbGRD b25maWd1cmF0aW9uOwogCQkJYnVpbGRTZXR0aW5ncyA9IHsKKwkJCQlERUJVR19JTkZPUk1BVElP Tl9GT1JNQVQgPSBkd2FyZjsKIAkJCQlHQ0NfRU5BQkxFX0NQUF9FWENFUFRJT05TID0gTk87CiAJ CQkJR0NDX0VOQUJMRV9DUFBfUlRUSSA9IE5POwogCQkJCUdDQ19FTkFCTEVfT0JKQ19FWENFUFRJ T05TID0gWUVTOwpJbmRleDogSmF2YVNjcmlwdEdsdWUvQ2hhbmdlTG9nCj09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT0KLS0t IEphdmFTY3JpcHRHbHVlL0NoYW5nZUxvZwkocmV2aXNpb24gMTU4NTMpCisrKyBKYXZhU2NyaXB0 R2x1ZS9DaGFuZ2VMb2cJKHdvcmtpbmcgY29weSkKQEAgLTEsMyArMSwxMCBAQAorMjAwNi0wOC0x NCAgTWFyayBSb3dlICA8b3BlbmRhcndpbi5vcmdAYmRhc2gubmV0Lm56PgorCisgICAgICAgIGh0 dHA6Ly9idWd6aWxsYS5vcGVuZGFyd2luLm9yZy9zaG93X2J1Zy5jZ2k/aWQ9MTAzODQKKyAgICAg ICAgQnVnIDEwMzg0OiBTd2l0Y2ggdG8gRFdBUkYgZm9yIFJlbGVhc2UgY29uZmlndXJhdGlvbgor CisgICAgICAgICogSmF2YVNjcmlwdEdsdWUueGNvZGVwcm9qL3Byb2plY3QucGJ4cHJvajoKKwog MjAwNi0wOC0wNCAgU2FtIFdlaW5pZyAgPHNhbS53ZWluaWdAZ21haWwuY29tPgogCiAgICAgICAg IFJldmlld2VkIGJ5IERhcmluLgpJbmRleDogSmF2YVNjcmlwdEdsdWUvSmF2YVNjcmlwdEdsdWUu eGNvZGVwcm9qL3Byb2plY3QucGJ4cHJvago9PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBKYXZhU2NyaXB0R2x1ZS9K YXZhU2NyaXB0R2x1ZS54Y29kZXByb2ovcHJvamVjdC5wYnhwcm9qCShyZXZpc2lvbiAxNTg1MSkK KysrIEphdmFTY3JpcHRHbHVlL0phdmFTY3JpcHRHbHVlLnhjb2RlcHJvai9wcm9qZWN0LnBieHBy b2oJKHdvcmtpbmcgY29weSkKQEAgLTQ1MSw2ICs0NTEsNyBAQAogCQkxNEFDNjYyRDA4Q0U3Nzkx MDA2OTE1QTggLyogUmVsZWFzZSAqLyA9IHsKIAkJCWlzYSA9IFhDQnVpbGRDb25maWd1cmF0aW9u OwogCQkJYnVpbGRTZXR0aW5ncyA9IHsKKwkJCQlERUJVR19JTkZPUk1BVElPTl9GT1JNQVQgPSBk d2FyZjsKIAkJCQlHQ0NfRU5BQkxFX0NQUF9FWENFUFRJT05TID0gTk87CiAJCQkJR0NDX0VOQUJM RV9DUFBfUlRUSSA9IE5POwogCQkJCUdDQ19FTkFCTEVfT0JKQ19FWENFUFRJT05TID0gWUVTOwpJ bmRleDogV2ViQ29yZS9DaGFuZ2VMb2cKPT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PQotLS0gV2ViQ29yZS9DaGFuZ2VMb2cJ KHJldmlzaW9uIDE1ODUzKQorKysgV2ViQ29yZS9DaGFuZ2VMb2cJKHdvcmtpbmcgY29weSkKQEAg LTEsMyArMSwxMCBAQAorMjAwNi0wOC0xNCAgTWFyayBSb3dlICA8b3BlbmRhcndpbi5vcmdAYmRh c2gubmV0Lm56PgorCisgICAgICAgIGh0dHA6Ly9idWd6aWxsYS5vcGVuZGFyd2luLm9yZy9zaG93 X2J1Zy5jZ2k/aWQ9MTAzODQKKyAgICAgICAgQnVnIDEwMzg0OiBTd2l0Y2ggdG8gRFdBUkYgZm9y IFJlbGVhc2UgY29uZmlndXJhdGlvbgorCisgICAgICAgICogV2ViQ29yZS54Y29kZXByb2ovcHJv amVjdC5wYnhwcm9qOgorCiAyMDA2LTA4LTEzICBNYWtzIE9ybG92aWNoICA8bWFrc2ltQGtkZS5v cmc+CiAKICAgICAgICAgUmV2aWV3ZWQgKGFuZCB0d2Vha2VkIGEgbGl0dGxlKSBieSBNYWNpZWou CkluZGV4OiBXZWJDb3JlL1dlYkNvcmUueGNvZGVwcm9qL3Byb2plY3QucGJ4cHJvago9PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09Ci0tLSBXZWJDb3JlL1dlYkNvcmUueGNvZGVwcm9qL3Byb2plY3QucGJ4cHJvagkocmV2aXNp b24gMTU4NTEpCisrKyBXZWJDb3JlL1dlYkNvcmUueGNvZGVwcm9qL3Byb2plY3QucGJ4cHJvagko d29ya2luZyBjb3B5KQpAQCAtNzQxNiw2ICs3NDE0LDcgQEAKIAkJMTQ5QzI4NDUwODkwMkIxMTAw OEE5RUZDIC8qIFJlbGVhc2UgKi8gPSB7CiAJCQlpc2EgPSBYQ0J1aWxkQ29uZmlndXJhdGlvbjsK IAkJCWJ1aWxkU2V0dGluZ3MgPSB7CisJCQkJREVCVUdfSU5GT1JNQVRJT05fRk9STUFUID0gZHdh cmY7CiAJCQkJRkVBVFVSRV9ERUZJTkVTID0gIlNWR19TVVBQT1JUIFhQQVRIX1NVUFBPUlQiOwog CQkJCUdDQ19FTkFCTEVfQ1BQX0VYQ0VQVElPTlMgPSBOTzsKIAkJCQlHQ0NfRU5BQkxFX0NQUF9S VFRJID0gTk87CkluZGV4OiBXZWJLaXQvQ2hhbmdlTG9nCj09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT0KLS0tIFdlYktpdC9D aGFuZ2VMb2cJKHJldmlzaW9uIDE1ODUzKQorKysgV2ViS2l0L0NoYW5nZUxvZwkod29ya2luZyBj b3B5KQpAQCAtMSwzICsxLDEwIEBACisyMDA2LTA4LTE0ICBNYXJrIFJvd2UgIDxvcGVuZGFyd2lu Lm9yZ0BiZGFzaC5uZXQubno+CisKKyAgICAgICAgaHR0cDovL2J1Z3ppbGxhLm9wZW5kYXJ3aW4u b3JnL3Nob3dfYnVnLmNnaT9pZD0xMDM4NAorICAgICAgICBCdWcgMTAzODQ6IFN3aXRjaCB0byBE V0FSRiBmb3IgUmVsZWFzZSBjb25maWd1cmF0aW9uCisKKyAgICAgICAgKiBXZWJLaXQueGNvZGVw cm9qL3Byb2plY3QucGJ4cHJvajoKKwogMjAwNi0wOC0xNCAgTWFjaWVqIFN0YWNob3dpYWsgIDxt anNAYXBwbGUuY29tPgogCiAgICAgICAgIFJ1YmJlciBzdGFtcGVkIGJ5IEJyYWR5LgpJbmRleDog V2ViS2l0L1dlYktpdC54Y29kZXByb2ovcHJvamVjdC5wYnhwcm9qCj09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT0KLS0tIFdl YktpdC9XZWJLaXQueGNvZGVwcm9qL3Byb2plY3QucGJ4cHJvagkocmV2aXNpb24gMTU4NTEpCisr KyBXZWJLaXQvV2ViS2l0Lnhjb2RlcHJvai9wcm9qZWN0LnBieHByb2oJKHdvcmtpbmcgY29weSkK QEAgLTE5MjMsNiArMTkyMyw3IEBACiAJCTE0OUMyODM0MDg5MDJCMEYwMDhBOUVGQyAvKiBSZWxl YXNlICovID0gewogCQkJaXNhID0gWENCdWlsZENvbmZpZ3VyYXRpb247CiAJCQlidWlsZFNldHRp bmdzID0geworCQkJCURFQlVHX0lORk9STUFUSU9OX0ZPUk1BVCA9IGR3YXJmOwogCQkJCUdDQ19D X0xBTkdVQUdFX1NUQU5EQVJEID0gZ251OTk7CiAJCQkJR0NDX0VOQUJMRV9DUFBfRVhDRVBUSU9O UyA9IE5POwogCQkJCUdDQ19FTkFCTEVfQ1BQX1JUVEkgPSBOTzsK