- 30 Oct, 2015 14 commits
-
-
Christian Schudt authored
-
Dave Cridland authored
Use StandardCharsets.UTF_8 with Reader/Writer classes.
-
Christian Schudt authored
This is a followup for fcf665a9.
-
Christian Schudt authored
-
Dave Cridland authored
Use real Java enums for presence subscription states.
-
Christian Schudt authored
The homemade IntEnum and Enum classes feel totally weird and confusing, when having java.lang.Enum around. This commit removes these classes. The static constants in the RosterItem class now feel superfluous, but are kept in there, because they are referenced by a lot of other classes and JSP pages.
-
Dave Cridland authored
Use Collections.singleton() where ArrayList has only one element.
-
Dave Cridland authored
Fix PluginClassLoader load resource path issue
-
Dave Cridland authored
Use try-with-resources statement to properly close resources.
-
Dave Cridland authored
Replace deprecated org.jivesoftware.util.Log with org.slf4j.Logger
-
Christian Schudt authored
-
Christian Schudt authored
-
Christian Schudt authored
- Not all resources were closed in finally block (see Launcher, PluginManager, UpdateManager, FaviconServlet, ...) - Get rid of verbose finally blocks and empty catch blocks, which makes the code more readable. - Generally, this is the more modern approach to deal with resources.
-
wmz7year authored
When load plugin resource files old commit will load all resource with / path Like /images/WEB-INF/web.xml old commit is /web.xml should be /images/WEB-INF/web.xml
-
- 29 Oct, 2015 14 commits
-
-
Dave Cridland authored
Deprecate ConcurrentHashSet in favor of Collections.newSetFromMap().
-
Christian Schudt authored
ConcurrentHashSet seems to be a relict from earlier Java times. Since Java 1.6 the preferred method to create a "ConcurrentHashSet" is to use Collections.newSetFromMap(new ConcurrentHashMap<?, ?>())
-
Christian Schudt authored
This has performance benefits. On my machine it's nearly twice as fast.
-
Dave Cridland authored
Add PluginClassLoader plugin loader isolation mechanism
-
Dave Cridland authored
fix getAuditedStartDate resetting startDate to null
-
Dave Cridland authored
Replace Arrays.asList() (with 1 arg) with Collections.singletonList().
-
Dave Cridland authored
Replace StringBuffer with StringBuilder (for performance).
-
Dave Cridland authored
Java Generics: Fix various "unchecked" compiler warnings.
-
Dave Cridland authored
Use Collections.unmodifiableCollection() instead of obsolete Immutabl…
-
Dave Cridland authored
Use integrated AWT SystemTray and TrayIcon classes instead of JDIC.
-
Dave Cridland authored
Make code more readable by using StandardCharsets.UTF_8.
-
Christian Schudt authored
-
Christian Schudt authored
- Use String.getBytes(Charset) instead of String.getBytes(String) - Use new String(byte[], Charset) instead of new String(byte[], String) -> Get rid of UnsupportedEncodingException. -> Less try-catch blocks.
-
Christian Schudt authored
Prefer StringBuilder.append(char) over append(String).
-
- 28 Oct, 2015 5 commits
-
-
Christian Schudt authored
JDIC is obsolete starting with JDK 1.6: "Most of the features provided by JDIC were incorporated into the JDK starting with version 1.6. As a result, the development of JDIC has come to an end."
-
Christian Schudt authored
-
Christian Schudt authored
-
Christian Schudt authored
Delete that class, it's not needed, since Java 5 Collections Framework.
-
Christian Schudt authored
This is the more correct API and also has very minor performance benefits.
-
- 27 Oct, 2015 2 commits
-
-
Christian Schudt authored
And don't use String concatenation within the append() method.
-
Christian Schudt authored
(for performance reasons)
-
- 23 Oct, 2015 1 commit
-
-
akrherz authored
-
- 22 Oct, 2015 4 commits
-
-
Guus der Kinderen authored
OF-946: Help Java to find the correct Security Provider
-
Guus der Kinderen authored
-
Guus der Kinderen authored
For some reason, the default validation fails to iterate over all providers and will fail if the default provider does not support the algorithm of the chain. To work around this issue, the code now iterates over each provider explicitly, returning success when at least one provider validates the chain successfully. This replaces an earlier attempt to fix this issue by explicitly naming one (and just one) provider.
-
Guus der Kinderen authored
OF-946: Openfire should allow for more than one set of key stores.
-