Env had a very old way of reading environment variables from
before Java supported environemnt variables from System. This
commit makes the code the same as core CDT's EnvironmentReader.
Env holds a copy of the EnvironmentReader code to avoid dependency of
terminal to other CDT.
Change-Id: I8929bf427af99d3d62cca28ee8090aa423c9aded
Signed-off-by: jantje <eclipse@baeyens.it>
The existing code had the assumption that >= 1 connector was installed
in the IDE. It is difficult to configure the IDE with no connectors as
the base terminal feature includes connectors, but it is technically
possible. So in that case display a simple error message and avoid
a hidden error (and stack trace to log).
Change-Id: Id17a1e5bff9aec251aa6f7fe6ff4e96cd2a01564
Using the mouse and not the focus listeners leads to lots
of weird, unintended consequences. For example:
- If mouse does not exit the control (because, e.g. it was not
in the control) then captureKeyEvents(false) is never issued
- If mouse does leave control, then control stops capturing
events. This means that although the cursor and focus is still
on the control, keyboard shortcuts no longer go to the control,
so Ctrl-C becomes copy again. This means, depending on where you
move the mouse changes the behaviour of the control.
This reverts commit 19351cbc2b.
Change-Id: I4f57c659e21d823df049b095159a34e5c110ef29
There are two ways of specifying fonts in the terminal,
one via font definitions, and one via the font directly. The
method via the font directly was already deprecated in the public
API, this commit follows that deprecation chain through to
make it easier to identify which code paths are not likely
relevant anymore.
These methods could simply be removed as part of the major version
bump.
Change-Id: I05d746c429c4e6c3ced5ee7bb7a6172e0a255c2d
GC.getForeground / Background always create a new Color object,
so this if expression was always true.
Change-Id: Ief4ccaf1adb70967301411b012c0474608f425cb
The inconsistency of lower case and upper case in the internal
representation of the colour names made it harder than it should be
to find where colours are used in the code.
Note that the SytleMap creates lower and uppercase entries for each
colour.
Change-Id: I16b4ebe4a97adecd01f835319aa0f0d175ae3bb6
And run save actions to modernize code a bit.
Change-Id: I4a718e73e8472679aa4ead7fdefe4259004ef6d6
Signed-off-by: Alexander Kurtakov <akurtako@redhat.com>
Bump BREE to Java 1.8 and fix various warnings as they are seen.
Change-Id: I3fb68f164832e6fa28c088c98ba57a7506365590
Signed-off-by: Alexander Kurtakov <akurtako@redhat.com>
It has been that org.eclipse.tm.terminal.view.feature and
org.eclipse.tm.terminal.feature shared the same feature plugin, which
meant they could not have their own branding.
Change-Id: I5bc48902c0ee93796fd4df7f847a39529bca3f7f
The 4.6.0 version marks the first version of terminal built in
CDT and in a different git repo - there are no API changes.
Change-Id: Ic6c4089d7e6ccbb2701bf59fedba2b6a8dd4c856
This is applying the per-project code formatting rules that would
be applied on save in the JDT editor
See also Bug 540373
Change-Id: Id04c972c3b3cb0a6ed1ab8581185359b7cf43f91
Replace simple uses of Iterator with a corresponding for-loop. Also add
missing braces on loops as necessary.
Change-Id: I7e158278fd2042b527f76b100815cf385b79ea20
Signed-off-by: chammer <carsten.hammer@t-online.de>
