[gjs] coverage: Use a local for fetchStatistics
- From: Jasper St. Pierre <jstpierre src gnome org>
- To: commits-list gnome org
- Cc:
- Subject: [gjs] coverage: Use a local for fetchStatistics
- Date: Mon, 3 Feb 2014 21:44:57 +0000 (UTC)
commit e9256b00a188c82aa8b8a383cc222f82a4b67abc
Author: Jasper St. Pierre <jstpierre mecheye net>
Date: Mon Feb 3 13:25:31 2014 -0500
coverage: Use a local for fetchStatistics
Rather than going half-way across the world for it...
modules/coverage.js | 18 ++++++------------
1 files changed, 6 insertions(+), 12 deletions(-)
---
diff --git a/modules/coverage.js b/modules/coverage.js
index 9c26d9a..b4ad9a8 100644
--- a/modules/coverage.js
+++ b/modules/coverage.js
@@ -679,32 +679,26 @@ function CoverageStatisticsContainer(files) {
* Debugger which in turn depends on objects injected in from another compartment */
function CoverageStatistics(files) {
this.container = new CoverageStatisticsContainer(files);
+ let fetchStatistics = this.container.fetchStatistics.bind(this.container);
/* 'debuggee' comes from the invocation from
* a separate compartment inside of coverage.cpp */
this.dbg = new Debugger(debuggee);
- this.dbg.fetchStatistics = function(statisticsContainer) {
- let container = statisticsContainer;
- return function (filename) {
- return container.fetchStatistics(filename);
- }
- } (this.container);
this.getNumberOfLinesFor = function(filename) {
- return this.container.fetchStatistics(filename).nLines;
+ return fetchStatistics(filename).nLines;
};
this.getExecutedLinesFor = function(filename) {
- return this.container.fetchStatistics(filename).expressionCounters;
+ return fetchStatistics(filename).expressionCounters;
};
this.getBranchesFor = function(filename) {
- return this.container.fetchStatistics(filename).branchCounters;
+ return fetchStatistics(filename).branchCounters;
};
this.getFunctionsFor = function(filename) {
- let functionCounters = this.container.fetchStatistics(filename).functionCounters;
-
+ let functionCounters = fetchStatistics(filename).functionCounters;
return _convertFunctionCountersToArray(functionCounters);
};
@@ -712,7 +706,7 @@ function CoverageStatistics(files) {
let statistics;
try {
- statistics = this.fetchStatistics(frame.script.url);
+ statistics = fetchStatistics(frame.script.url);
} catch (e) {
/* We don't care about this frame, return */
return undefined;
[
Date Prev][
Date Next] [
Thread Prev][
Thread Next]
[
Thread Index]
[
Date Index]
[
Author Index]