Adam Gent

[FR] - Limit recursion to a specific depth

1

Mustache allows recursive partials and JMustache supports this but for many this can be dangerous especially if JMustache is used to render user templates. Currently I believe the only depth...

JUL bridge and System.Logger are inconsistent on handling of OFF and ALL

9

When logging with either the System.Logger or the JUL the expectation of passing `Level.OFF` (which does not have an SLF4J analog) is inconsistent when they should be same. Ideally they...

https://github.com/hsingh/java-shortuuid/blob/2a4d72f195a3900ef86492173a51f6cad9f25969/src/main/java/com/hsingh/shortuuid/ShortUuid.java#L64 The `25d` should be `256d`.

After a maven refresh annotation processors cannot find resources

3

After a maven refresh annotation processors cannot find resources in the classpath (ie files in `src/main/resources`). Eclipse 4.12 and m2e-apt 1.5.2 This might be related to: https://github.com/jbosstools/m2e-apt/issues/67 The original bug...

I sort of rely on the profileUrl being set. I haven't tried this but it should work. org.springframework.social.foursquare.connect.FoursquareAdapter.setConnectionValues(Foursquare, ConnectionValues) ``` public void setConnectionValues(Foursquare foursquare, ConnectionValues values) { FoursquareUser user =...

I could not get the tycho compiler version to work at all. This is for others but this works for me: ```xml eclipse org.apache.maven.plugins maven-compiler-plugin eclipse ${parent.root}/etc/eea/prefs/org.eclipse.jdt.core.prefs CLASSPATH ${parent.root}/etc/eea true...

#55 ConfigProviderResolver lazy initialization

1

The `ConfigProviderResolver` should not initialize on just being imported for a variety of reasons. The major reasons is makes initialization less predictable and bootstrapping more complicated. Performance is also a...

The `ConfigProvider` (EDIT on name... apologies) has a static initializer. For static initialization I usually use this pattern See https://en.wikipedia.org/wiki/Initialization-on-demand_holder_idiom to avoid unnecessary loading. Thus ``` private static final ConfigProviderResolver...

The original variable expansion `${SomeVariable}` was powered by commons lang `StrSubstitutor` prior `0.2.3` (see #14) . The new variable expansion is a simpler and in theory faster albeit probably buggy....

.insert() should not return void.

1

Currently to insert an object in JIRM you do: ``` dao.insert(object); ``` This is inconsistent with the fluent portion of the API and leaves no room for extensibility. It should...