Merge pull request #12 from launchdarkly/drichelson/ch3424/sdk-migration-to-2-2-3-failing-for-java-7

Add no-arg constructors for things we deserialize from json.

Author: drichelson

src/main/java/com/launchdarkly/client/FeatureFlag.java

@@ -16,17 +16,17 @@ class FeatureFlag {
   private static final Type mapType = new TypeToken<Map<String, FeatureFlag>>() {
   }.getType();
 
-  private final String key;
-  private final int version;
-  private final boolean on;
-  private final List<Prerequisite> prerequisites;
-  private final String salt;
-  private final List<Target> targets;
-  private final List<Rule> rules;
-  private final VariationOrRollout fallthrough;
-  private final Integer offVariation; //optional
-  private final List<JsonElement> variations;
-  private final boolean deleted;
+  private String key;
+  private int version;
+  private boolean on;
+  private List<Prerequisite> prerequisites;
+  private String salt;
+  private List<Target> targets;
+  private List<Rule> rules;
+  private VariationOrRollout fallthrough;
+  private Integer offVariation; //optional
+  private List<JsonElement> variations;
+  private boolean deleted;
 
   static FeatureFlag fromJson(String json) {
     return LDConfig.gson.fromJson(json, FeatureFlag.class);
@@ -36,6 +36,9 @@ static Map<String, FeatureFlag> fromJsonMap(String json) {
     return LDConfig.gson.fromJson(json, mapType);
   }
 
+  // We need this so Gson doesn't complain in certain java environments that restrict unsafe allocation
+  FeatureFlag() {}
+
   FeatureFlag(String key, int version, boolean on, List<Prerequisite> prerequisites, String salt, List<Target> targets, List<Rule> rules, VariationOrRollout fallthrough, Integer offVariation, List<JsonElement> variations, boolean deleted) {
     this.key = key;
     this.version = version;

src/main/java/com/launchdarkly/client/Prerequisite.java

@@ -1,8 +1,11 @@
 package com.launchdarkly.client;
 
 class Prerequisite {
-  private final String key;
-  private final int variation;
+  private String key;
+  private int variation;
+
+  // We need this so Gson doesn't complain in certain java environments that restrict unsafe allocation
+  Prerequisite() {}
 
   Prerequisite(String key, int variation) {
     this.key = key;

src/main/java/com/launchdarkly/client/Rule.java

@@ -8,7 +8,12 @@
  * Invariant: one of the variation or rollout must be non-nil.
  */
 class Rule extends VariationOrRollout {
-  private final List<Clause> clauses;
+  private List<Clause> clauses;
+
+  // We need this so Gson doesn't complain in certain java environments that restrict unsafe allocation
+  Rule() {
+    super();
+  }
 
   Rule(List<Clause> clauses, Integer variation, Rollout rollout) {
     super(variation, rollout);

src/main/java/com/launchdarkly/client/Target.java

@@ -3,8 +3,11 @@
 import java.util.List;
 
 class Target {
-  private final List<String> values;
-  private final int variation;
+  private List<String> values;
+  private int variation;
+
+  // We need this so Gson doesn't complain in certain java environments that restrict unsafe allocation
+  Target() {}
 
   Target(List<String> values, int variation) {
     this.values = values;

src/main/java/com/launchdarkly/client/VariationOrRollout.java

@@ -13,8 +13,11 @@
 class VariationOrRollout {
   private static final float long_scale = (float) 0xFFFFFFFFFFFFFFFL;
 
-  private final Integer variation;
-  private final Rollout rollout;
+  private Integer variation;
+  private Rollout rollout;
+
+  // We need this so Gson doesn't complain in certain java environments that restrict unsafe allocation
+  VariationOrRollout() {}
 
   VariationOrRollout(Integer variation, Rollout rollout) {
     this.variation = variation;
@@ -56,8 +59,11 @@ private float bucketUser(LDUser user, String key, String attr, String salt) {
   }
 
   static class Rollout {
-    private final List<WeightedVariation> variations;
-    private final String bucketBy;
+    private List<WeightedVariation> variations;
+    private String bucketBy;
+
+    // We need this so Gson doesn't complain in certain java environments that restrict unsafe allocation
+    Rollout() {}
 
     Rollout(List<WeightedVariation> variations, String bucketBy) {
       this.variations = variations;
@@ -66,8 +72,11 @@ static class Rollout {
   }
 
   static class WeightedVariation {
-    private final int variation;
-    private final int weight;
+    private int variation;
+    private int weight;
+
+    // We need this so Gson doesn't complain in certain java environments that restrict unsafe allocation
+    WeightedVariation() {}
 
     WeightedVariation(int variation, int weight) {
       this.variation = variation;