Skip to content

Commit

Permalink
found another location where the file watcher is turned on
Browse files Browse the repository at this point in the history
  • Loading branch information
benfry committed Sep 11, 2014
1 parent f95ed92 commit bd8cf86
Show file tree
Hide file tree
Showing 2 changed files with 13 additions and 9 deletions.
19 changes: 10 additions & 9 deletions app/src/processing/app/Editor.java
Expand Up @@ -3,6 +3,7 @@
/*
Part of the Processing project - http://processing.org
Copyright (c) 2012-14 The Processing Foundation
Copyright (c) 2004-12 Ben Fry and Casey Reas
Copyright (c) 2001-04 Massachusetts Institute of Technology
Expand Down Expand Up @@ -2586,16 +2587,16 @@ public boolean handleSaveAs() {
statusNotice(Language.text("editor.status.saving"));
try {
if (sketch.saveAs()) {
//a saveAs moves where the files are, so a listener must be attached to the new location
initFileChangeListener();
// Disabling for 3.0a4
if (false && Preferences.getBoolean("editor.watcher")) {
// "Save As" moves where the files are, so a listener must be
// attached to the new location.
// TODO shouldn't this remove the old listener?
initFileChangeListener();
}
// statusNotice("Done Saving.");
// status is now printed from Sketch so that "Done Saving."
// is only printed after Save As when progress bar is shown.

// Disabling this for 0125, instead rebuild the menu inside
// the Save As method of the Sketch object, since that's the
// only one who knows whether something was renamed.
//sketchbook.rebuildMenusAsync();
// status is now printed from Sketch so that "Done Saving."
// is only printed after Save As when progress bar is shown.
} else {
statusNotice(Language.text("editor.status.saving.canceled"));
return false;
Expand Down
3 changes: 3 additions & 0 deletions build/shared/revisions.txt
Expand Up @@ -17,6 +17,9 @@ until all those issues are sorted out.
+ The OS X default File menu (shown when no windows are open) now has the
order/naming changes found in the sketch window File menu.

+ Turning off file watching because of errant "this sketch has changed"
messages. Hopefully this will return soon.
https://github.com/processing/processing/issues/2852

[ bug fixes ]

Expand Down

0 comments on commit bd8cf86

Please sign in to comment.