diff options
| author | Mark Spruiell <mes@zeroc.com> | 2004-04-20 19:14:18 +0000 |
|---|---|---|
| committer | Mark Spruiell <mes@zeroc.com> | 2004-04-20 19:14:18 +0000 |
| commit | a1a113e35ab9098f0377be77d94879ded41270d7 (patch) | |
| tree | 377a092f091b92b382443e1860a3fe876123dc20 /java/src | |
| parent | fix (diff) | |
| download | ice-a1a113e35ab9098f0377be77d94879ded41270d7.tar.bz2 ice-a1a113e35ab9098f0377be77d94879ded41270d7.tar.xz ice-a1a113e35ab9098f0377be77d94879ded41270d7.zip | |
more facet fixes
Diffstat (limited to 'java/src')
| -rw-r--r-- | java/src/Ice/ObjectPrxHelperBase.java | 5 | ||||
| -rw-r--r-- | java/src/IceInternal/Reference.java | 12 |
2 files changed, 15 insertions, 2 deletions
diff --git a/java/src/Ice/ObjectPrxHelperBase.java b/java/src/Ice/ObjectPrxHelperBase.java index aeebbe0c39d..4b0f86e3011 100644 --- a/java/src/Ice/ObjectPrxHelperBase.java +++ b/java/src/Ice/ObjectPrxHelperBase.java @@ -246,6 +246,11 @@ public class ObjectPrxHelperBase implements ObjectPrx public final ObjectPrx ice_newFacet(String newFacet) { + if(newFacet == null) + { + newFacet = ""; + } + if(newFacet.equals(_reference.facet)) { return this; diff --git a/java/src/IceInternal/Reference.java b/java/src/IceInternal/Reference.java index 8a51a34980b..6f6cbdedf9d 100644 --- a/java/src/IceInternal/Reference.java +++ b/java/src/IceInternal/Reference.java @@ -133,7 +133,7 @@ public final class Reference // // For compatibility with the old FacetPath. // - if(facet == null || facet.length() == 0) + if(facet.length() == 0) { s.writeStringSeq(null); } @@ -189,7 +189,7 @@ public final class Reference s.append(id); } - if(facet != null && facet.length() > 0) + if(facet.length() > 0) { // // If the encoded facet string contains characters which @@ -834,6 +834,14 @@ public final class Reference Ice.ObjectAdapter rvAdapter, boolean collocationOpt) { + // + // Validate string arguments. + // + assert(ident.name != null); + assert(ident.category != null); + assert(fac != null); + assert(adptId != null); + // // It's either adapter id or endpoints, it can't be both. // |