eclipse / tm4e

TextMate support in Eclipse IDE
https://projects.eclipse.org/projects/technology.tm4e
Eclipse Public License 2.0
89 stars 56 forks source link

Don't report errors to be consistent with the rest of the class #662

Closed iloveeclipse closed 8 months ago

iloveeclipse commented 8 months ago

Fixes https://github.com/eclipse/tm4e/issues/661

sebthom commented 8 months ago

Do you think instead of swallowing all exceptions we could just ignore logging if a filenotfoundexception is thrown? or log with debug level?

iloveeclipse commented 8 months ago

Do you think instead of swallowing all exceptions we could just ignore logging if a filenotfoundexception is thrown? or log with debug level?

We could, I have even started with that, but this would mean the exceptions shouldn't be handled by the "low level" code as of today and propagated back to callers (which have context and can sort out if they want / won't report an error).

But that would be exact the opposite of what was done in 0.9.0 and the effort isn't justified.

In general, looking on similar code in my workspace I see that most of the "content probe" code mutes any errors anyway as it very seldom that an "probe error" is interesting for anyone except the bundle code owner.

So I've finally decided to go along with the rest of the class (which ignores errors) and be consistent :-)

iloveeclipse commented 8 months ago
[ERROR] Failed to execute goal org.eclipse.tycho:tycho-packaging-plugin:4.0.4:validate-version (default-validate-version) on project org.eclipse.tm4e.ui: Unqualified OSGi version 0.9.1.qualifier must match unqualified Maven version 0.9.0-SNAPSHOT for SNAPSHOT builds -> [Help 1]

I don't see what is expected now.

sebthom commented 8 months ago

I'm looking into it right now.

sebthom commented 8 months ago

can you give this a try? https://github.com/eclipse/tm4e/pull/663