Mercurial > dive4elements > river
comparison artifacts/src/main/java/org/dive4elements/river/artifacts/model/DataFacet.java @ 8853:8c64617a7991
Some source code comments and fixme's
| author | gernotbelger |
|---|---|
| date | Thu, 18 Jan 2018 18:33:52 +0100 |
| parents | af13ceeba52a |
| children | 3f49835a00c3 |
comparison
equal
deleted
inserted
replaced
| 8852:8f6d6d26e96f | 8853:8c64617a7991 |
|---|---|
| 102 /** | 102 /** |
| 103 * Return a deep copy. | 103 * Return a deep copy. |
| 104 */ | 104 */ |
| 105 @Override | 105 @Override |
| 106 public Facet deepCopy() { | 106 public Facet deepCopy() { |
| 107 // FIXME: why not use the full constructor instead? would also fix the next problem | |
| 107 DataFacet copy = new DataFacet(); | 108 DataFacet copy = new DataFacet(); |
| 109 // FIXME: usage of internal knowledge of parent class... | |
| 110 // Either the set method should be correctly overwritten, or implement a correct copy-constructor! | |
| 108 copy.set(this); | 111 copy.set(this); |
| 109 copy.type = type; | 112 copy.type = type; |
| 110 copy.hash = hash; | 113 copy.hash = hash; |
| 111 copy.stateId = stateId; | 114 copy.stateId = stateId; |
| 112 return copy; | 115 return copy; |
| 113 } | 116 } |
| 114 } | 117 } |
| 115 // vim:set ts=4 sw=4 si et sta sts=4 fenc=utf-8 : |