Bug 151331

Summary: Removed useless variable scriptResult
Product: WebKit Reporter: Matthew Daiter <mdaiter>
Component: JavaScriptCoreAssignee: Matthew Daiter <mdaiter>
Status: RESOLVED FIXED    
Severity: Trivial CC: achristensen, commit-queue, jonlee, keith_miller, mark.lam, mdaiter, msaboff, saam, webkit.review.bot
Priority: P3 Keywords: EasyFix, PlatformOnly
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch none

Matthew Daiter
Reported 2015-11-16 16:30:46 PST
Inside of testExecutionTimeLimit(), there's a variable called scriptResult that uses a variable that never actually gets read, only written to. This can be removed, unless there's some sort of benchmark result that needs a write for a fair testExecutionTimeLimit() execution.
Attachments
Patch (3.92 KB, patch)
2015-11-16 16:33 PST, Matthew Daiter 		no flags
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Matthew Daiter
Comment 1 2015-11-16 16:33:12 PST
Created attachment 265637 [details] Patch
WebKit Commit Bot
Comment 2 2015-11-17 11:56:05 PST
Comment on attachment 265637 [details] Patch Clearing flags on attachment: 265637 Committed r192520: <http://trac.webkit.org/changeset/192520>
WebKit Commit Bot
Comment 3 2015-11-17 11:56:08 PST
All reviewed patches have been landed. Closing bug.
Note You need to log in before you can comment on or make changes to this bug.