Skip to content
This repository has been archived by the owner on Sep 6, 2021. It is now read-only.

Commit

Permalink
Merge pull request #7332 from adobe/zaggino/stat-changed-path
Browse files Browse the repository at this point in the history
Interim fix for #6803: don't warn when Node-side watcher stat fails
  • Loading branch information
peterflynn committed Mar 27, 2014
2 parents bcab0a2 + 647428d commit 4776a53
Showing 1 changed file with 3 additions and 2 deletions.
5 changes: 3 additions & 2 deletions src/filesystem/impls/appshell/AppshellFileSystem.js
Original file line number Diff line number Diff line change
Expand Up @@ -34,7 +34,7 @@ define(function (require, exports, module) {
NodeDomain = require("utils/NodeDomain");

var FILE_WATCHER_BATCH_TIMEOUT = 200; // 200ms - granularity of file watcher changes

/**
* Callback to notify FileSystem of watcher changes
* @type {?function(string, FileSystemStats=)}
Expand Down Expand Up @@ -91,7 +91,8 @@ define(function (require, exports, module) {
if (needsStats) {
exports.stat(path, function (err, stats) {
if (err) {
console.warn("Unable to stat changed path: ", path, err);
// warning has been removed due to spamming the console - see #7332
// console.warn("Unable to stat changed path: ", path, err);
return;
}
_changeCallback(path, stats);
Expand Down

0 comments on commit 4776a53

Please sign in to comment.