1
0
mirror of https://github.com/arduino/Arduino.git synced 2024-12-05 16:24:12 +01:00
Commit Graph

1534 Commits

Author SHA1 Message Date
Cristian Maglie
cf54cce660 SplashScreenHelper output progress in console for text-only mode
This commit makes this changes:

- SplashScreenHelper is now local in Base constructor
- if SplashScreenHelper is instantiated with a null SplashScreen
  instance then it outputs progress in console and avoid to make
  calls to Swing toolkit
- The parsing of command line arguments is anticipated so we can
  determine if we are in command line or GUI mode early and setup
  objects that produces output to not use graphics toolkits.
- In this case the SplashScreenHelper is initialized with a real
  splashscreen only if we are in GUI mode
2016-11-25 15:04:24 +01:00
Cristian Maglie
fbe6bf3792 Moved all initializations into Base constructor 2016-11-25 15:04:24 +01:00
Cristian Maglie
27d1b8d9a1 Merge branch 'windows-dpi-awareness' 2016-11-21 12:43:01 +01:00
Cristian Maglie
41092be596 Merge branch 'editor-refactor-fixup' 2016-11-09 14:25:45 +01:00
Cristian Maglie
650840381f Small cleanup, no code change 2016-11-07 14:51:25 +01:00
Cristian Maglie
5b4af28f47 Removed dead code 2016-11-07 14:47:33 +01:00
Cristian Maglie
10dcc1d932 When a new tab is created is also selected
Previously it was selected always the last tab because the action
sequence was:

 - create the new tab (in the last position)
 - select the new tab index (last)
 - sort the tabs (the new tab is now in the middle but the selected
   is always the last)

instead the correct action sequence is

 - create the new tab (in the last position)
 - sort the tabs (now the new tab is in the middle)
 - select the new tab index (now the correct index is selected)
2016-11-07 12:25:53 +01:00
Cristian Maglie
5244daa5d1 Use the same Comparator to sort EditorTab and SketchFile 2016-11-07 12:13:30 +01:00
Cristian Maglie
d63162b5a1 Added Platform.getSystemDPI() API 2016-11-04 10:23:48 +02:00
Martino Facchin
1d74bb87ac Trigger Keywords update if a new library is added
Fixes #5415
2016-10-27 12:47:00 +02:00
Martino Facchin
c6642c1bd2 Reorder tabs when a new file is inserted/renamed
Fixes #5402
2016-10-27 12:47:00 +02:00
Martino Facchin
4e84238527 Restore "Delete" tab functionality
The file was being deleted but the tab was still there
2016-10-27 12:47:00 +02:00
Cristian Maglie
3d6abacee4 Merge branch 'master' of git://github.com/duff2013/Arduino 2016-10-18 16:39:30 +02:00
duff2013
9a3881ecce set grid color through theme.txt now 2016-10-18 00:04:43 -07:00
Cristian Maglie
a8525c6bb0 Parse libraries index after updating it (when launche from CLI)
Fix #5424
2016-10-04 14:06:32 +02:00
duff2013
db8526c27c upgrade Serial Plotter 2016-10-03 19:34:57 -07:00
xloem
8b5a900ae1 Message parsing logic fix
comparison was invalid if some parts were not parsable numbers
2016-09-30 17:04:01 -04:00
xloem
be01c39c3a Empty graph hack fixes #4722 and fixes #5316 2016-09-30 16:56:57 -04:00
Cristian Maglie
bd7e76533a Removed redundant type specifiers for generics 2016-09-29 20:15:56 +02:00
Cristian Maglie
fca863f325 Removed unused variable 2016-09-29 20:15:55 +02:00
Cristian Maglie
3cc9446f04 Removed unused import 2016-09-29 20:15:55 +02:00
Cristian Maglie
0def10443f Fixed another error in string comparison 2016-09-29 20:15:55 +02:00
Cristian Maglie
635807105a Merge remote-tracking branch 'cmaglie/examples-menu-take2' 2016-09-21 09:53:50 +02:00
Cristian Maglie
c58e2e68d1 Fix regression: ignore case in file extensions
Fix #5389
2016-09-20 17:34:03 +02:00
Cristian Maglie
646c9f798f Fix regression: after "save as..." window title is updated
Fix #5388
2016-09-20 16:36:41 +02:00
Cristian Maglie
1efa07f010 Enable visibility of selection on find
Fix #5380
2016-09-20 15:31:17 +02:00
Cristian Maglie
c5a6a44b55 Simplify FindReplace.find() logic (part 2)
The snippet:

    boolean wrapNeeded = false;
    if (wrap && nextIndex == -1) {
      // if wrapping, a second chance is ok, start from the end
      wrapNeeded = true;
    }

Can be moved inside the `if (nextIndex == -1)` that follows, this way:

    if (nextIndex == -1) {
      boolean wrapNeeded = false;
      if (wrap) {
        // if wrapping, a second chance is ok, start from the end
        wrapNeeded = true;
      }

      [...CUT...]

      if (wrapNeeded) {
        nextIndex = backwards ? text.lastIndexOf(search) : text.indexOf(search, 0);
      }
    }

but since `wrapNeeded` is used only at the very end of the `if` statement
we can move it forward:

    if (nextIndex == -1) {
      [...CUT...]

      boolean wrapNeeded = false;
      if (wrap) {
        // if wrapping, a second chance is ok, start from the end
        wrapNeeded = true;
      }
      if (wrapNeeded) {
        nextIndex = backwards ? text.lastIndexOf(search) : text.indexOf(search, 0);
      }
    }

and finally simplify it by removing `wrapNeeded` altogether:

    if (nextIndex == -1) {
      [...CUT...]

      if (wrap) {
        nextIndex = backwards ? text.lastIndexOf(search) : text.indexOf(search, 0);
      }
    }
2016-09-20 13:58:36 +02:00
Cristian Maglie
47fcff77d5 Simplify FindReplace.find() logic (part 1)
The snippet:

    boolean wrapNeeded = false;
    if (wrap && nextIndex == -1) {
      // if wrapping, a second chance is ok, start from the end
      wrapNeeded = true;
    }

is present on both sides of the `if` statement so it can be factored out.
2016-09-20 13:52:24 +02:00
Cristian Maglie
9723726387 Call ensureOffsetNotInClosedFold only when a match is found 2016-09-20 13:45:11 +02:00
Cristian Maglie
096e545257 Examples of arch-independent libs from Arduino goes into Examples for any board
The other cases remain unchanged
2016-09-19 16:30:18 +02:00
Cristian Maglie
821ee53d40 Replaced some example menu labels
From: `Examples from Built-in Libraries`
To:   `Examples for any board`

From: `Examples from Arduino AVR Boards Libraries` (selected platform)
To:   `Examples for Arduino/Genuino Micro` (selected board)

From: `Examples from Arduino AVR Boards Libraries` (referenced platform)
To:   `Examples for Arduino AVR Boards` (referenced platform)
2016-09-19 16:27:47 +02:00
PaulStoffregen
afc7c596fc Only sort ideLibs if it's not empty 2016-09-19 14:25:25 +02:00
PaulStoffregen
d49401bdbf Improve Examples menu 2016-09-19 14:25:13 +02:00
Matthijs Kooijman
c927237912 Fix tab order in FindReplace
When searching through all tabs, the order was accidentally reversed.
This was broken by commit d2bac86 (Remove tab switching logic from
Sketch).

This also fixes a problem where "replace all" would only work on the
first and last tab (since it would search backwards from the first tab
to the last tab and then conclude it was done).

This fixes a part of #5380.
2016-09-19 12:05:11 +02:00
Cristian Maglie
9abf25d222 Fix indentation and license 2016-08-31 19:10:01 +02:00
Martino Facchin
b5223cc576 Ensure that the selected code is not folded during "Search and Replace"
Fixes #4845
2016-08-26 16:42:44 +02:00
Martino Facchin
300b1d7290 Fix rebase errors 2016-08-26 16:42:44 +02:00
Martino Facchin
29e79c8157 update API to v1 2016-08-26 16:42:44 +02:00
Martino Facchin
efc07a1c0e Add Cloud API integration
This method discovers if a connected board needs an additional core and helps the user downloading it via BoardManager
2016-08-26 16:42:44 +02:00
Martino Facchin
f96f1d321a add overloaded method to NotificationPopup to tune autoclose 2016-08-26 16:42:44 +02:00
Matthijs Kooijman
a78a3ed6b5 Update comment
This comment still talked about Processing related stuff which doesn't
really apply anymore.
2016-08-26 16:42:44 +02:00
Matthijs Kooijman
94754d7841 In Base.handleOpen(), compare Files rather than Strings
Comparing a File object automatically takes care of filesystem case
sensitivity, whereas strings do not, so this makes the comparison
slightly more reliable.
2016-08-26 16:42:44 +02:00
Matthijs Kooijman
4f0af2af57 Store the build path used in Sketch
Previously, everywhere where it was needed, the path was requested from
BaseNoGui. Because the path is based on a hash of the sketch filename,
every caller would get the same path for the same sketch.

However, it makes more sense to store the path used for a given sketch
inside the Sketch object. This prevents having to pass around or
regenerate the build path everywhere, and no longer requires the build
path to be deterministic (though it still is in this commit).

This allows removing some methods and constructors of which two versions
were available - one with a build path argument and one without.
2016-08-26 16:42:44 +02:00
Matthijs Kooijman
1d15c434dc Let SketchFile.delete() call Sketch.removeFile()
Previously, callers of `SketchFile.delete()` would also call
`Sketch.removeFile()`, but letting SketchFile handle this is more
robust.

This is possible now that SketchFile keeps a reference to Sketch and
makes updating the Sketch file list less fragile.

Eventually this might be further decoupled by letting SketchFile
broadcast a "deleted" event instead.
2016-08-26 16:42:44 +02:00
Matthijs Kooijman
aae0bc4c47 Rename SketchFile.deleteFile() to delete()
The extra "File" in the name was a bit redundant, and this is more
consistent with `Sketch.delete()`.
2016-08-26 16:42:44 +02:00
Matthijs Kooijman
ccec28ea45 Move sketch deletion from SketchController into Sketch
This isn't much code, but it makes deletion more consistent with
renaming and saving with the SketchController handling the UI part and
Sketch actually doing the delete.
2016-08-26 16:42:44 +02:00
Matthijs Kooijman
85d48394a4 Merge Sketch.renameFileTo() into SketchFile.renameTo()
Now that SketchFile keeps a reference to its Sketch,
`SketchFile.renameTo()` can call `Sketch.checkNewFilename()`, so there
is no need for the renaming itself to go through Sketch.

This changes the parameter for `SketchFile.renameTo()` from File to
String, to enforce that only the filename is changed, not the directory
name.
2016-08-26 16:42:44 +02:00
Matthijs Kooijman
b28f1a479d Remove unused Base.listFiles() methods 2016-08-26 16:42:44 +02:00
Matthijs Kooijman
c0f41cae5d Remove Base.removeDir() and Base.removeDescendants()
These methods shouldn't really be in Base (or BaseNoGui, which did the
actual work), especially since there is already a
`FileUtils.recursiveDelete()` which just does the same thing. This
commit removes the code from Base and BaseNoGui and instead uses the
method from FileUtils.

There is one difference between these methods: the Base methods did not
delete files if the "compiler.save_build_files" preference was set.
However, the Base methods were only used when deleting a sketch, or
deleting an existing folder before overwriting it on save as, so this
preference didn't actually do what it was supposed to anyway, so
dropping it shouldn't be a problem.
2016-08-26 16:42:44 +02:00
Matthijs Kooijman
72f815bcf9 Refactor file adding and renaming, and save as handling
This commits replaces a significant part of the code handling these
features. A lot of responsibilities are moved from SketchController to
Sketch, though the code involved is rewritten mostly.

Most of the handling now happens inside Sketch, including various checks
against the new filename. Basically SketchController processes the user
input to decide what needs to be done, and Sketch checks if it can be
done and does it.

If problems occur, an IOException is thrown, using a translated error
message that is shown by SketchController as-is. This might not be the
best way to transfer error messages (regular IOExceptions might contain
less-friendly messages), so this might need further improvement later.

In addition to moving around code and responsibilities, this code also
changes behaviour in some places:
 - Because Sketch and SketchFile are now in control of renames and
   saves, they can update their internal state after a rename. This
   removes the need for reloading the entire sketch after a rename or
   save as and allows `Editor.handleOpenUnchecked()` to be removed.
 - When renaming the entire sketch, all files used to be saved before
   renaming, since the sketch would be re-opened after renaming. Since
   the re-opening no longer happens, there is no longer a need to save
   the sketch, so any unsaved changes remain unsaved in the editor after
   renaming the sketch.
 - When renaming or adding new files, duplicate filenames are detected.
   Initially, this happened case sensitively, but it was later changed to
   use case insensitive matching to prevent problems on Windows (where
   filenames cannot differ in just case). To prevent complexity, this
   did not distinguish between systems. In commit 5fbf9621f6 (Sketch
   rename: allowig a case change rename if NOT on windows), the
   intention was to only do case insensitive checking on Windows, but it
   effectively disabled all checking on other systems, making the check
   not catch duplicate filenames at all.

   With this commit, all these checks are done using `File.equals()`
   instead of comparing strings, which is already aware of the case
   sensitivity of the platform and should act accordingly.
 - Some error messages were changed.
 - When adding a file, an empty file is not created directly, but only a
   SketchFile and EditorTab is added. When the sketch is saved, the file
   is created.
 - When importing a file that already exists (thus overwriting it),
   instead of replacing the SketchFile instance, this just lets the
   EditorTab reload its contents. This was broken since the introduction
   of EditorTab. The file would be replaced, but not this was not
   reflected in the editor, which is now fixed. This change allows
   `Sketch.replaceFile()` to be removed.
 - When importing a file that does not exist yet (thus adding it), a tab
   is now also added for it (in addition to a SketchFile). This was
   broken since the introduction of EditorTab, and would result in the
   file being added, but not shown in the editor.

This commit adds a `Sketch.renameFileTo()` method, to rename a single
file within the sketch. It would be better to integrate its contents
into `Sketch.renameTo()`, but that does not have access to the `Sketch`
instance it is contained in. This will be changed in a future commit.
2016-08-26 16:42:44 +02:00
Matthijs Kooijman
57a237752d Clean up pde to ino renaming
This makes a few related changes:
 - `FileUtils.replaceExtension()` is introduced to handle replacing the
   .pde extension with .ino.
 - Instead of iterating .pde files on disk, this iterates SketchFiles in
   memory, saving another lookup from filename -> SketchFile later.
 - `SketchController.renameCodeToInoExtension()` is removed. Now it no
   longer needs to look up the SketchFile and FileUtils handles the
   extension replacement, this method did not have any reason to exist
   anymore.
 - Instead of hardcoding the .pde extension, a new
   Sketch.OLD_SKETCH_EXTENSIONS constant is introduced.
2016-08-26 16:42:44 +02:00
Matthijs Kooijman
98c0e0f841 Remove Base.copyDir()
There was already a nearly identical `FileUtils.copy()` that copies
directories recursively.

The only difference is that now hidden files *are* copied, but version
control files (according the list in FileUtils) are not. Since this only
affects the copying of the "data" directory during save as, this
should not be much of a problem.
2016-08-26 16:42:44 +02:00
Matthijs Kooijman
7a73d0b3db Change Compiler.pathToSketch from String to File
Keeping filenames as File objects for as long as possible is generally a
good idea and this removes a dependency on `Sketch.getMainFilePath()`,
so it can be removed later.
2016-08-26 16:42:44 +02:00
Matthijs Kooijman
959fba7865 Let Sketch.getPrimaryFile return a SketchFile
Previously, it returned a File object, which the Sketch separately
stored from the primary SketchFile. By letting it just return the
SketchFile, and let callers query that for the filename, Sketch does not
need to store the File object itself and there is less chance of info
getting out of sync.
2016-08-26 16:42:44 +02:00
Matthijs Kooijman
2d367e757d Allow .cpp files named after the primary .ino file
This limitation was added a long time ago, when the build system did not
cope with this. The current build system handles this situation just
fine, so this limitation can be lifted.
2016-08-26 16:42:44 +02:00
Matthijs Kooijman
c5c3a771e5 Remove Base.handle*Replace()
These methods dealt with opening up a sketch inside an existing editor
window, but they were unused - all sketches open in a new window now.
2016-08-26 16:42:44 +02:00
Matthijs Kooijman
7d277b359c Remove Editor.stopHandler
This handler was only always assigned the DefaultStopHandler, which did
nothing. It was called in a few places, but since it never does
anything, better remove it. For properly supporting stopping of external
processes, some better architecture should be added instead.
2016-08-26 16:42:44 +02:00
Matthijs Kooijman
8e0d007ce2 Rename SketchCode to SketchFile
That name more accurately reflects its purpose: It represents a single
file within a sketch. This just updates the class name and variable
names referring to these objects and some comments, so no behaviour
should change.
2016-08-26 16:42:44 +02:00
Matthijs Kooijman
dfa60dacc1 Store a SketchCode instance in RunnerException
Previously, the index of the SketchCode instance in the list kept by
Sketch was kept, which isn't really robust.

With this change, Sketch.indexOfCode is no longer needed and is removed.
2016-08-26 16:42:44 +02:00
Matthijs Kooijman
a07a9ff895 Move SketchController.prepareDataFolder() to Sketch 2016-08-26 16:42:44 +02:00
Matthijs Kooijman
a4e2e80c1c Let importLibrary use Sketch.SKETCH_EXTENSIONS
For determining if the current file was a sketch file, it previously
(indirectly) used a hardcoded "ino" comparison. Now, it uses
`SKETCH_EXTENSIONS` so it also applies to .pde files and the hardcoded
"ino" (and the methods leading up to it) can be removed.
2016-08-26 16:42:44 +02:00
Matthijs Kooijman
5e68b66c5e Clean up SketchController.nameCode a bit
This lets it use FileUtils.splitFilename and reference Sketch.EXTENSIONS
and the new Sketch.DEFAULT_SKETCH_EXTENSION directly, allowing to remove
a few helper functions.
2016-08-26 16:42:44 +02:00
Matthijs Kooijman
b3ad637f8c Simplify SketchController.addFile using FileUtils.hasExtension 2016-08-26 16:42:44 +02:00
Matthijs Kooijman
d70900e5ad Let Sketch.getPrettyName() hide extension for .ino and .pde only
Before, `getPrettyName()` would return the extension-less name for all
files. There were a lot of places that checked for .ino and/or .pde
files and and called `getPrettyName()` for those, and `getFileName()`
for others. By moving this check into `getPrettyName()`, all those
callers become more simple, and more consistent (there were 5 different
checks to basically achieve the same thing).

There are small changes in behaviour, where .pde is now also hidden but
was not before. Also, the print header now shows extensions for other
files, which makes it more consistent with the tab names. For cases
where the old behaviour was still required, `Sketch.getBaseName()` was
added.

At the same time, the actual handling of the filenames is simplified by
using methods from FileUtils.

With this change `Sketch.getFileNameWithExtensionIfNotIno()` and
`SketchController.getHiddenExtensions()` are no longer needed and are
removed.
2016-08-26 16:42:44 +02:00
Matthijs Kooijman
5792ec212b Remove Editor.findTabIndex(String), which is now unused 2016-08-26 16:42:44 +02:00
Matthijs Kooijman
8afeb501cb In SketchController.nameCode, only switch tabs when adding file
When renaming a file, the current tab is already the correct
one, so no need to switch. This allows looking up the tab index based on
the SketchCode object, instead of doing a filename lookup.
2016-08-26 16:42:44 +02:00
Matthijs Kooijman
fa82fe4db1 Do not find a tab based on filename, when we have a SketchCode
Filename-based matching is a bit more fragile, so just do a lookup based
on the SketchCode object we already have instead.
2016-08-26 16:42:44 +02:00
Matthijs Kooijman
3d573a7e86 Use SketchCode.isPrimary() in more places
This should be more reliable than comparing filenames or assuming the
primary file is always at index 0.
2016-08-26 16:42:44 +02:00
Matthijs Kooijman
dd5c1787fd Move isModified() from SketchController to Sketch
Also, update any code that uses it, removing the dependency on
SketchController entirely if possible.
2016-08-26 16:42:44 +02:00
Matthijs Kooijman
629953e20d Rename Sketch and SketchData classes
Sketch is now called SketchController, since it didn't really represent
a sketch, but just handled the GUI-related stuff for a given sketch
(note that it is not strictly a controller in the MVC-sense, but it does
have a similar function). SketchData more accurately represented the
actual sketch, so it is now called Sketch. Below, the new names are
used.

Editor now keeps both a current Sketch and SketchController object, and
the Sketch object is created by Editor and passed to SketchController,
instead passing a File and letting SketchController create the Sketch.
Wherever possible, code now uses the Sketch directly (or indirectly,
through the new `SketchController.getSketch()`) and the accessors in
SketchController that merely forwarded to Sketch have been removed.
There are few things that now live in SketchController but should be
moved to Sketch (`isModified()`, `isUntitled()`), so some of the code
still has a dependency on SketchController that should be removed later.

This commit mostly renames classes, methods and variables, it should not
change the behaviour in any way.
2016-08-26 16:42:44 +02:00
Matthijs Kooijman
eba1098efb Delete Sketch.prepare()
The only remaining thing that the method did was call `ensureExistence()`.
However, a call the `prepare()` was always followed by a call to
`build()`, which already calls `ensureExistence()`, so `prepare()`
didn't have any remaining value.
2016-08-26 16:42:44 +02:00
Matthijs Kooijman
f3d8ba219f Remove support for a "code" folder in sketches
When adding a file to a sketch (using drag and drop, or the Sketch ->
Add file... menu item), .o, .a and .so files would be saved into a
"code" subdirectory of the sketch. This seems to be a remnant of
processing, where also .dll and .jar files could be added to a sketch to
be used. In the Arduino IDE, these code files serve no special purpose,
and are not treated specially, so it makes no sense to keep this code
around.

One implication of this is that when "save as" is used, a "code"
subdirectory is no longer copied, which might affect people using this
"code" subdirectory for other purposes.

Similarly, there is support for a "data" subdirectory, in which all
other files (that are not sketch source files) are stored, and which is
also copied on "save as". Support for this folder is kept intact, since
this appears occasionally used (the ESP8266 project uses it to store and
upload additional data files, for example).

This change was discussed on the mailing list in the "Anyone using
"data" and "code" subdirectories in sketches?" thread:
https://groups.google.com/a/arduino.cc/forum/#!msg/developers/zPlraPq55ho/ejrLqITnAgAJ
2016-08-26 16:42:44 +02:00
Matthijs Kooijman
76be212f23 Remove applet.html handling
This was a remnant of Processing, this file has no special meaning for
an Arduino sketch, so this code can just be removed.
2016-08-26 16:42:44 +02:00
Matthijs Kooijman
1d21378a5f Replace requestFocus() by requestFocusInWindow() where applicable
The former gives focus to the window in which a component is present,
while the latter only changes the focus within the current window (not
focusing the window itself if it is not focused yet). Java documentation
recommends changing `requestFocusInWindow()` where possible, due to some
platform-dependent behaviour in `requestFocus()`.

When focusing the serial monitor and plotter, `requestFocus()` is still
used, since then the focused window *should* change.
2016-08-26 16:42:44 +02:00
Matthijs Kooijman
f57b90c1c8 Remove code that preserves caret position during auto format
`EditorTab.setText()` now already preserves the caret position. The code
used during auto-format tried a bit harder to preserve the position
correctly, and probably worked better in a few specific cases, but for
most cases they would both end up approximating the caret position
anyway. To make the code simpler, better just stick to the simpler
approach.
2016-08-26 16:42:44 +02:00
Matthijs Kooijman
6558c0654e Remove all highlights in EditorTab.setText()
When completely replacing all text, all highlights will end up at the
start of the file. Since keeping them at the right place is tricky (even
impossible in some circumstances), just remove them now.

This already happened in the autoformat code, so that part can be
removed.
2016-08-26 16:42:44 +02:00
Matthijs Kooijman
c08fd76cad Remove UndoManager stuff from AStyle
The only change that happens is a single `setText()` call, which already
results in a single undo action.
2016-08-26 16:42:44 +02:00
Matthijs Kooijman
31284feb9b Merge SketchData.sortCodes() into addCode()
By now, all calls to `addCode()` were followed by a call to
`sortCodes()`, and it seems like a task for SketchData to keep its list
sorted. Previously, this separation made some sense, since `addCode()`
was also used while loading a sketch, and you would only want to sort
once. Now, sketch loading uses a SortedSet, so this is no longer a
requirement.
2016-08-26 16:42:44 +02:00
Matthijs Kooijman
8725bb1ec4 Clean up sketch loading
Previously, the Sketch constructor called its `load()` function, which
called the `SketchData.load()` function to load files and then
`Editor.sketchLoaded()` to initialize the GUI with the loaded files.
When external editing was enabled, `Sketch.load()` was called again
when activating the Arduino app, to reload the entire sketch.

With this commit, the `Sketch.load()` function is removed, and
`SketchData.load()` is called from the SketchData constructor. Instead
of Sketch calling `Editor.sketchLoaded()`, that method is renamed
to `createTabs()` and called by `Editor.HandleOpenInternal()` directly
after creating the Sketch object.

Handling of external editor mode has also changed. When the Arduino
application is activated, instead of fully reloading the sketch (through
the now-absent `Sketch.load()` method), the new `SketchData.reload()`
method is called to reload the list of files in the sketch. If it
changed, all tabs are re-created. If not, only the current tab is
reloaded. When the user switches from one tab to another, that tab is
also reloaded. This ensures that the visible  tab is always up-to-date,
without needlessly reloading all tabs all the time. When external
editing mode is enabled or disabled, all tabs are reloaded too, to make
sure they are up-to-date.

When re-creating all tabs, no attempt is made to preserve the currently
selected tab. Since adding or removing files happens rarely, this should
not be a problem. When files are changed, the currently selected tab is
implicitly preserved (because the tab is reloaded, not recreated). The
caret (and thus scroll) position is preserved by temporarily changing
the caret update policy, so the caret does not move while the text is
swapped out. This happens in `EditorTab.setText()` now, so other callers
can also profit from it.

To support checking for a changed list of files in
`SketchData.reload()`, a `SketchCode.equals()` method is added, that
just checks if the filenames are equal. Additionally, the loading of the
file list for a sketch has now moved from `SketchData.load()` to
`SketchData.listSketchFiles()`, so `reload()` can also use it. At the
same time, this loading is greatly simplified by using a sorted Set and
`FileUtils.listFiles()`.

In external editor mode, to ensure that during compilation the version
from disk is always used instead of the in-memory version, EditorTab
detaches itself from its SketchCode, so SketchCode has no access to the
(possibly outdated) in-memory contents of the file.
2016-08-26 16:42:44 +02:00
Matthijs Kooijman
283ccc150d Print errors while reloading externally edited files
These exceptions were silently dropped, which is pretty much never a
good idea.
2016-08-26 16:42:44 +02:00
Matthijs Kooijman
6715f41c0e Let SketchCode track if it is the primary file
This makes checking for the primary file easier, without having to know
the index of a file in the list of tabs, or relying on the fact that the
primary file is always first (it still is, though).

This changes some places in Sketch to use the new
`SketchCode.isPrimary()` method, but there probably are a lot more
places in the code that could be start to use it as well.
2016-08-26 16:42:44 +02:00
Matthijs Kooijman
6c2a94ecc5 Remove SketchCodeDocument
This class served no purpose anymore, so it can be removed. The
`SketchCode.getMetadata()` and `setMetaData()` methods only served to
keep track of a SketchCodeDocument instance (and were no longer used),
so these are removed too, just like some SketchCode constructors dealing
with this metadata object.
2016-08-26 16:42:44 +02:00
Matthijs Kooijman
d2bac8659e Remove tab switching logic from Sketch
This lets all code directly call `Editor.selectTab()`, or the newly
introduced `Editor.selectNextTab()` or `Editor.selectPrevTab()`. This
also adds a new `Editor.findTabIndex(String)` to look up a tab based on
the filename (what `Sketch.setCurrentCode(String)` used to do). At some
point, this method might need to be removed, but for now it allows other
code to keep working with minimal changes.
2016-08-26 16:42:44 +02:00
Matthijs Kooijman
0764eb7b19 Remove current and currentIndex variables from Sketch
Instead of letting Sketch (also) keep track of the currently selected
tab, this moves the responsibility to Editor instead. When Sketch need
to know the current tab and file, it now asks Editor.

Switching between tabs is still handled through Sketch methods, but that
will be cleaned up later.
2016-08-26 16:42:44 +02:00
Matthijs Kooijman
6b31cffaec Add getTabs() and getCurrentTabIndex() to Editor and use them
Previously, some of the GUI code would use Editor.getSketch() to get the
current sketch, and Sketch.getCurrentCode() to find out the currently
selected tab. Since this code is really concerned with the currently
open tab in the GUI, it makes more sense to query the Editor tabs list
directly.

This removes all references the current sketch code, as tracked by
Sketch, external to Sketch itself. This prepares for removing the
current tab tracking from Sketch later.
2016-08-26 16:42:44 +02:00
Matthijs Kooijman
ca573351bb Do not store file contents in SketchCode
Now that each file in the sketch has its own text area in the GUI, it is
no longer needed to store the (possibly modified) contents of each file
inside SketchCode. Keeping the contents in the text area is sufficient.
Doing so allows removing the code that dealt with copying contents from
the text area into the SketchCode instance at the right time, which was
fragile and messy.

However, when compiling a sketch, the current (modified) file contents
still should be used. To allow this, the TextStorage interface is
introduced. This is a simple interface implemented by EditorTab, that
allows the SketchCode class to query the GUI for the current contents.
By using an interface, there is no direct dependency on the GUI code. If
no TextStorage instance is attached to a SketchCode, it will just assume
that the contents are always unmodified and the contents from the file
will be used during compilation.

When not using the GUI (e.g. just compiling something from the
commandline), there is no need to load the file contents from disk at
all, the filenames just have to be passed to arduino-builder and the
compiler. So, the SketchCode constructor no longer calls its `load()`
function, leaving this to the GUI code to call when appropriate. This
also modifies the `SketchCode.load()` function to return the loaded
text, instead of storing it internally.

To still support adding new files to a sketch (whose file does not
exist on disk yet), the EditorTab constructor now allows an initial
contents to be passed in, to be used instead of loading from disk. Only
the empty string is passed for new files now, but this could also be
used for the bare minimum contents of a new sketch later (which is now
down by creating a .ino file in a temporary directory).

Another side effect of this change is that all changes to the contents
now happen through the text area, which keeps track of modifications
already. This allows removing all manual calls to `Sketch.setModified()`
(even more, the entire function is removed, making `Sketch.isModified()`
always check the modification status of the contained files).
2016-08-26 16:42:44 +02:00
Matthijs Kooijman
422c111d81 Let EditorTab listen for changes to the text area
Previously, EditorTab set the Document on the SketchCodeDocument, and
the latter would listen for changes, only forwarding the modified status
to SketchCode. This commit cuts out a step and lets EditorTab call
SketchCode::setModified directly.

Additionally, the DocumentTextChangedListener helper class is added,
which wraps a simple (lambda) function to be called whenever anything
about the document text is modified. This hides the verbosity of having
to handle both insertion and deletion, and instead suffices with just
having a single lambda function instead.
2016-08-26 16:42:44 +02:00
Matthijs Kooijman
982d4f3fbf Use a separate RSyntaxTextArea for each editor tab
RSyntaxTextArea appears to support using a single instance and replacing
the underlying text and document when switching between tabs, but in
practice this support is not complete and even though the
RSyntaxTextArea developers did some work to improve the situation, they
recommend to just use a seperate instance for each tab.

This commit implements exactly that. A new class EditorTab is introduce
to wrap the RSyntaxTextArea and containing scroll pane, and to
encapsulate the code related to handling the text area itself. Doing so
removes some quirks and prepares for some later additions. In
particular, error highlights are now no longer shared between all tabs,
which was previously the case.

This commit mostly moves code from Editor into EditorTab, and updates
the callers to use getCurrentTab() and call methods on the result
instead of calling them on Editor. Some code is added to take care of
creating multiple EditorTab objects and switching between them. Some
small changes have been made to make the flow of opening files work,
though these are mostly a bit hacky.

While moving code, changes to the rest of the code were kept minimal,
retaining existing interfaces as much as possible. This sometimes result
in less than ideal code, which should be cleaned up in subsequent
commits.

The SketchCodeDocument class has been pretty much emptied out, since
it was mostly used to store things for tabs in the background, which are
now just stored in each RSyntaxTextArea separately. The last remaining
bits of this class can probably be moved or implemented differently
later, so it can be removed.

The entire flow of working with sketches and files needs to be cleaned
up next, so no thorough attempt at testing this commit was done. It is
likely that there are plenty of corner cases and race conditions, which
will be fixed once the reset of the code is cleaned up.

Fixes #3441
2016-08-26 16:42:44 +02:00
Matthijs Kooijman
c945b6c30a Let EditorToolbar use the global KeyboardFocusManager
For some toolbar buttons, when it is clicked while shift is pressed, its
function changes. When handling the click event, this information is
directly taken from KeyEvent.isShiftDown(). However, to also show the
proper tooltip *before* clicking, EditorToolbar listened to key events
on the main text area, to know when shift is (not) pressed.

This approach means that pressing shift while the text area is not
focused will not change the tooltip, and creates some unwanted coupling
between the toolbar and the text area.

This commit changes this approach to instead use the global
KeyboardFocusManager. Any key presses pass through there before being
dispatched to the currently focused component, so this makes sure that
any shift presses are caught, as well as making EditorToolbar a bit more
self-contained.
2016-08-26 16:42:44 +02:00
Cristian Maglie
d6db1deef0 Update revision log 2016-08-17 17:24:04 +02:00
Cristian Maglie
59e7c9ef60 Revert "Improve Examples menu"
This reverts commit 7235f9dc15.
2016-08-17 17:08:21 +02:00
Cristian Maglie
b583697e0b Revert "Only sort ideLibs if it's not empty"
This reverts commit 9b324bc9cc.
2016-08-17 17:08:14 +02:00
Cristian Maglie
4c6d2f4a82 Added new languages ach, kk and te 2016-08-16 13:16:38 +02:00
Cristian Maglie
aea77c889d Fixed some simple warnings 2016-08-16 12:52:24 +02:00
Cristian Maglie
eee54f1b3a Merge branch 'examplesmenu' of git://github.com/PaulStoffregen/Arduino into HEAD 2016-08-12 17:21:24 +02:00
Cristian Maglie
7008f6c57c ContributionsIndexer now has bundled hardware path as a field 2016-08-11 10:57:05 +02:00
PaulStoffregen
9b324bc9cc Only sort ideLibs if it's not empty 2016-08-02 02:43:04 -07:00
PaulStoffregen
7235f9dc15 Improve Examples menu 2016-07-28 16:58:28 -07:00
Sandeep Mistry
1a6be715ab Merge pull request #4794 from facchinm/upload_fail_handling
Handling gracefully upload failure
2016-07-20 10:18:57 -04:00
Sandeep Mistry
afe204f97f Merge pull request #4828 from facchinm/issue_4762
Filter examples based on contributed libraries by architecture
2016-07-19 10:00:18 -04:00
Cristian Maglie
8f20f4d98b "Include library" now checks for includes property
A new property "includes" has been added to library.properties.
This property contains a comma-separated list of the files to be included when
the user selects the "Include library" command on the Arduino IDE.

If the property is missing the old behaviour is used.
2016-06-23 13:04:56 +02:00
Martino Facchin
93578cef42 Fix ssh authentication on newer ssh clients
For Console monitor on network ports
2016-04-15 10:00:41 +02:00
Cristian Maglie
60b169fbe9 Merge branch 'ctrlDeleteBehavior' of https://github.com/avargas-nearsoft/Arduino 2016-04-07 11:32:46 +02:00
Martino Facchin
f6880fe617 Merge pull request #4792 from facchinm/solve_serial_windows
Rework serial ports handling
2016-04-06 18:03:45 +02:00
Arturo Vargas
a70bab3ea0 Add suport for OSX (ALT+DELETE) 2016-04-06 08:25:39 -07:00
Martino Facchin
c28c854936 Filter examples based on contributed libraries by architecture
Solves #4762
2016-04-06 14:41:12 +02:00
Cristian Maglie
f74afc4ad9 Remove incorrect Ctrl+K / Ctrl+W bindings
for a better explanation see @nopdotcom comment here:
https://github.com/arduino/Arduino/pull/4743#issuecomment-201447876
2016-04-06 12:18:45 +02:00
Jay Carlson
02914d79b9 Add more of OS X's native Emacs keybindings
OS X supports many simple Emacs keybindings in native text fields. (It
can do this because the system UI uses Cmd instead of Ctrl as its main
modifier key.) Add bindings to functions which already exist in
DefaultEditorKit.

I have a Ctrl-k (cut to end of line) implementation, but it is an
additional static RecordableTextAction class. I have not included it.
2016-04-06 12:17:58 +02:00
Cristian Maglie
050e8ba304 Merge branch 'cut/copyEnabled' of https://github.com/avargas-nearsoft/Arduino 2016-04-06 12:09:04 +02:00
Arturo Vargas
3efcc9ec48 Change the CTRL+DELETE behavior to restore the next word 2016-04-04 14:42:40 -07:00
Ivan Esparza
9c583829de Fixes issue 4440 2016-04-04 16:53:46 +02:00
Cristian Maglie
0c453355f4 Merge branch 'fix-annoying-popup' 2016-04-04 13:34:14 +02:00
Cristian Maglie
8d7ee63c70 Autoclose notification popup after 10 seconds. 2016-04-04 13:30:42 +02:00
Martino Facchin
84fdb0c0c3 Remove stray debug print 2016-04-01 17:05:54 +02:00
Martino Facchin
84ebc68af7 Handle case with empty error message 2016-04-01 12:06:32 +02:00
Martino Facchin
63de1cccfb Avoid generating an exception if upload fails
The current method of reporting upload errors is based on an exoteric combination of exceptions which makes return error code useless
The Uploader.java message() implementation is too avrdude-dependant to allow easy portability since the upload tools are becoming a lot and very different

With this commit we try to avoid exceptions and only use the external uploader's exit code to decide the status bar message.
The message can be:
- the last line containing "error" string (any case) or
- the usual avrdude message parsing (to keep compatibility with translations)

Needs testing with all platform and all supported upload tools
2016-04-01 12:06:21 +02:00
Martino Facchin
52ef55380d avoid NPE if serial monitor is waiting too long for opening 2016-04-01 11:38:53 +02:00
Martino Facchin
821c665460 Change message if requiring infos from Network port 2016-04-01 11:38:53 +02:00
Martino Facchin
243fc68763 Rework Serial ports handling and add Board info menu
This commit introduces the concept of stateful board list (vs. original stateless) and board serial number.

The board is now an "entity" composed by the triplet port/vid/pid. These informations come from libListSerial "light" function. When the board list changes, it triggers a request for the additional infos to libListSerial. These information contains the serial number of the boards.

These brings a lighter and faster scanning process. Some logic has been introduced to handle a board with the S/N only exposed in the bootloader (like 32u4).
In this case the disappearing port acquires the bootloader's S/N

A menu (under Ports menu) shows the currently connected port info and can be used for bugreporting
2016-04-01 11:38:53 +02:00
Martino Facchin
6d5597b070 Avoid multiple concurrent compile/upload operations
Disable Compile/Run buttons as they get press, and reenable only on function exit.
The launched upload process has now a 2minutes timeout before being terminated forcefully.
10 second after pressing "Upload" the button comes pressable again, but this time the previous upload command gets killed explicitely
2016-04-01 11:38:52 +02:00
Martino Facchin
b99ab40ba2 Enable links to directly open Lib/Board manager
If a sketch contains a link to http://librarymanager/${dropdown}#${filter} or http://boardmanager/${dropdown}#${filter} (for example http://librarymanager/All#OneWire) Library or Board manager will be opened applying the secified filters
2016-04-01 10:55:35 +02:00
Martino Facchin
a329731525 use hi-res icons for serial monitor and plotter
fixes #4701
2016-04-01 10:55:04 +02:00
Arturo Vargas
baa8a6e8a3 Disable cut and copy if there is not selected text 2016-03-31 15:11:20 -07:00
Cristian Maglie
1a7eaaf8b2 Fix always-on-top notification popup 2016-03-31 18:49:44 +02:00
Ivan Esparza
bbf909fd4f Changed color of version text in About box 2016-03-22 12:28:33 -07:00
agdl
057bf6391d This fixes #4572 2016-02-29 09:26:40 +01:00
Cristian Maglie
b02ce54387 Removed leftover stub for testing NotificationPopup dialog 2016-02-01 10:18:41 +01:00
Cristian Maglie
8e4ce4cfd1 Serial monitor font is scaled too 2016-02-01 10:14:19 +01:00
Cristian Maglie
5642c291d7 Fixed default initial editor size 2016-01-26 17:11:53 +01:00
Cristian Maglie
639d991dad Some small cosmetics 2016-01-26 17:11:53 +01:00
Cristian Maglie
57c6d096f6 Notification pop-up is now correctly scaled 2016-01-26 16:48:14 +01:00
Cristian Maglie
4659c6f985 Auto-scale editor and console fonts as well 2016-01-26 16:48:14 +01:00
Cristian Maglie
59ec660c9b Fixed minimum size for a bunch of GUI elements 2016-01-26 12:10:12 +01:00
Cristian Maglie
f23577499f Split pane is now sized correctly 2016-01-26 12:10:12 +01:00
Cristian Maglie
9215c06676 Installers windows are now correctly scaled. 2016-01-26 12:10:12 +01:00
Cristian Maglie
2f0b0db5ba Set maximum scale to 400% 2016-01-26 12:09:57 +01:00
Cristian Maglie
409f3161c8 Added GUI scaling in preference panel. 2016-01-26 12:09:57 +01:00
Cristian Maglie
08ad60032f Fix for some font rendering problems on linux 2016-01-26 12:09:57 +01:00
Cristian Maglie
3928b6a04f Fixed a small mess in EditorHeader 2016-01-26 12:09:57 +01:00
Cristian Maglie
d920c06699 Now the IDE use vectorial images whenever possible
The caller of Theme.getThemeImage(...) now pass only the name of
the needed resource and the theme folder is searche in the following
order:

  - name.svg
  - name.png (if svg is not available)
  - name@2x.png (if none of the above are available or if 1x png is
    too low resolution for the current scaling factor)
2016-01-26 12:09:57 +01:00
Cristian Maglie
e2fec88f5f Added some missing Theme.scale(..)
Now toolbar and header are correctly rendered
2016-01-26 12:09:56 +01:00
Cristian Maglie
967153fe87 Factored out scaling formula into an helper method
Also use a default value of 100 in case "gui.scalePercent" is not set.
2016-01-26 12:09:56 +01:00
Cristian Maglie
f239f5b5ab Moving getLibImage and getThemeImage into Theme class 2016-01-26 12:09:56 +01:00
Cristian Maglie
f1fc862575 Added hi-res toolbar images
Hi-resolution images are saved with the "@2x.png" suffix, the image
loader will select the best image available based on the user selected
scaling.

Missing hi-res images can be added later together with lo-res images.
2016-01-26 12:09:56 +01:00
Cristian Maglie
5637130818 Converted toolbar icons to png 2016-01-26 12:09:55 +01:00
Cristian Maglie
08d35106c2 Add scaling for themed fonts and graphics (hires displays) (Paul Stoffregen) 2016-01-26 12:09:55 +01:00
Cristian Maglie
c3d2bbdb86 Merge branch 'save-on-close' 2016-01-25 17:57:21 +01:00
Cristian Maglie
0bceb93b7c When closing the last sketch, the window position is saved as default
This allows the use-case with users editing one sketch at a time, that
seems to be the most common scenario:

 1. User position the editor as desired
 2. User close the IDE
 3. User opens the IDE by double clicking on another .ino file
 4. The IDE is opened again at the same position

See #4432
2016-01-22 18:08:49 +01:00
Cristian Maglie
61f11f60e1 Factored out logic to retrieve editor placement
The check for "resolution-changed" is performed when an editor
location is retrieved from preferences. This commit rationalize
access to PreferencesData and prepares for the next improvement.
2016-01-22 18:06:09 +01:00
Cristian Maglie
f5f478ced6 Factored out method to save sketch location 2016-01-22 18:06:08 +01:00