summaryrefslogtreecommitdiff
path: root/gitweb
diff options
context:
space:
mode:
authorJakub Narebski <jnareb@gmail.com>2011-05-27 13:49:59 (GMT)
committerJunio C Hamano <gitster@pobox.com>2011-05-27 18:00:35 (GMT)
commit4510165934eba7be94dea822e2fb1cfa89e70ca3 (patch)
tree22edea642169da904ef7435ad3a0ad11afdd8deb /gitweb
parent2e987f92408f5ddd20246d7d9553b57d90767d54 (diff)
downloadgit-4510165934eba7be94dea822e2fb1cfa89e70ca3.zip
git-4510165934eba7be94dea822e2fb1cfa89e70ca3.tar.gz
git-4510165934eba7be94dea822e2fb1cfa89e70ca3.tar.bz2
gitweb.js: No need for inProgress in blame_incremental.js
JavaScript is single-threaded, so there is no need for protection against re-entrancy via inProgress variable. In particular calls to setInterval handler are stacked if handler doesn't finish before new interrupt (before new interval). The same happens with events - they are (hopefully) stacked if even handler didn't finish work. Signed-off-by: Jakub Narebski <jnareb@gmail.com> Signed-off-by: Junio C Hamano <gitster@pobox.com>
Diffstat (limited to 'gitweb')
-rw-r--r--gitweb/static/js/blame_incremental.js21
1 files changed, 3 insertions, 18 deletions
diff --git a/gitweb/static/js/blame_incremental.js b/gitweb/static/js/blame_incremental.js
index 676da6b..4841805 100644
--- a/gitweb/static/js/blame_incremental.js
+++ b/gitweb/static/js/blame_incremental.js
@@ -420,8 +420,6 @@ function handleLine(commit, group) {
// ----------------------------------------------------------------------
-var inProgress = false; // are we processing response
-
/**#@+
* @constant
*/
@@ -536,7 +534,7 @@ function processData(unprocessed, nextReadPos) {
*
* @param {XMLHttpRequest} xhr: XMLHttpRequest object
*
- * @globals pollTimer, commits, inProgress
+ * @globals pollTimer, commits
*/
function handleError(xhr) {
errorInfo('Server error: ' +
@@ -544,8 +542,6 @@ function handleError(xhr) {
clearInterval(pollTimer);
commits = {}; // free memory
-
- inProgress = false;
}
/**
@@ -553,7 +549,7 @@ function handleError(xhr) {
*
* @param {XMLHttpRequest} xhr: XMLHttpRequest object (unused)
*
- * @globals pollTimer, commits, inProgress
+ * @globals pollTimer, commits
*/
function responseLoaded(xhr) {
clearInterval(pollTimer);
@@ -561,15 +557,13 @@ function responseLoaded(xhr) {
fixColorsAndGroups();
writeTimeInterval();
commits = {}; // free memory
-
- inProgress = false;
}
/**
* handler for XMLHttpRequest onreadystatechange event
* @see startBlame
*
- * @globals xhr, inProgress
+ * @globals xhr
*/
function handleResponse() {
@@ -609,13 +603,6 @@ function handleResponse() {
return;
}
- // in case we were called before finished processing
- if (inProgress) {
- return;
- } else {
- inProgress = true;
- }
-
// extract new whole (complete) lines, and process them
while (xhr.prevDataLength !== xhr.responseText.length) {
if (xhr.readyState === 4 &&
@@ -633,8 +620,6 @@ function handleResponse() {
xhr.prevDataLength === xhr.responseText.length) {
responseLoaded(xhr);
}
-
- inProgress = false;
}
// ============================================================