126771 2014-01-10 12:18:05 -0800 Web Inspector: use c++11 features in web-only agents 2014-08-01 14:19:38 -0700 1 1 1 Unclassified WebKit Web Inspector 528+ (Nightly build) All All RESOLVED FIXED InRadar P2 Normal --- 1 bburg joepeck joepeck timothy webkit-bug-importer oldest_to_newest 966502 0 bburg 2014-01-10 12:18:05 -0800 This has been annoying me. :) 966506 1 webkit-bug-importer 2014-01-10 12:22:42 -0800 <rdar://problem/15794772> 966514 2 timothy 2014-01-10 12:30:07 -0800 Joe fixed InspectorAgent in bug 126763 when he moved it to JSC. No need to do that one, you will just conflict. 966541 3 bburg 2014-01-10 13:15:10 -0800 Joe is handling this when moving stuff around to JSC. 966542 4 joepeck 2014-01-10 13:17:42 -0800 Yes, as I've been moving code down into JSC I've been making it sparkle. So nullptr, unique_ptr, some references, better whitespace, etc. There is plenty of code that code be updated in WebCore though. InstrumentingAgents, any WebCore agent (so not Runtime/Debugger). I'll happily review a patch making these changes. 966545 5 joepeck 2014-01-10 13:18:28 -0800 Another big one that is annoying me: OVERRIDE, FINAL 966593 6 joepeck 2014-01-10 14:20:07 -0800 Another big one that is annoying me: ASCIILiteral 969334 7 joepeck 2014-01-17 17:14:25 -0800 With Anders help I was about to get clang-modernize working. With that we can automate adding "override" where it is needed and using "nullptr". It makes a bunch of changes: Transform: AddOverride - Accepted: 1488 Transform: UseNullptr - Accepted: 267 Though I should first let Brian's more involved Reference change land first. 969983 8 joepeck 2014-01-20 13:30:59 -0800 Fixed nullptr and override with: <https://webkit.org/b/127206> Modernize WebCore/inspector with override <https://webkit.org/b/127303> Modernize WebCore/inspector with nullptr 1026333 9 joepeck 2014-08-01 14:19:38 -0700 Pretty sure this is fixed, right? nullptr, override, final, lots of unique_ptr. We can file new bugs about other things (ASCIILiteral, References).