Skip to content

Commit 5681967

Browse files
Deserialize only if structs exist
1 parent 7331617 commit 5681967

6 files changed

Lines changed: 23 additions & 19 deletions

File tree

lib/utils/deserialization/capnpQueryDeSerializer/capnpQueryDeSerializer.js

Lines changed: 5 additions & 5 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

lib/utils/deserialization/capnpQueryDeSerializer/capnpQueryDeSerializer.js.map

Lines changed: 1 addition & 1 deletion
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

lib/utils/serialization/capnpQuerySerializer/capnpQuerySerializer.js

Lines changed: 4 additions & 2 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

lib/utils/serialization/capnpQuerySerializer/capnpQuerySerializer.js.map

Lines changed: 1 addition & 1 deletion
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

src/utils/deserialization/capnpQueryDeSerializer/capnpQueryDeSerializer.ts

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -77,9 +77,9 @@ const capnpQueryDeSerializer = (buffer: ArrayBuffer | ArrayBufferLike) => {
7777
layout: query.getLayout(),
7878
status: query.getStatus(),
7979
type: query.getType(),
80-
writer: deserializeWrite(query),
81-
reader: deserializeQueryReader(query.getReader()),
82-
array: deserializeArray(query.getArray()),
80+
writer: query.hasWriter() ? deserializeWrite(query) : undefined,
81+
reader: query.hasReader() ? deserializeQueryReader(query.getReader()) : undefined,
82+
array: query.hasArray() ? deserializeArray(query.getArray()) : undefined,
8383
totalFixedLengthBufferBytes: query
8484
.getTotalFixedLengthBufferBytes()
8585
.toNumber(),
@@ -88,8 +88,8 @@ const capnpQueryDeSerializer = (buffer: ArrayBuffer | ArrayBufferLike) => {
8888
varOffsetsMode: query.getVarOffsetsMode(),
8989
varOffsetsAddExtraElement: query.getVarOffsetsAddExtraElement(),
9090
varOffsetsBitsize: query.getVarOffsetsBitsize(),
91-
config: deserializeConfig(query.getConfig()),
92-
stats: deserializeStats(query.getStats()),
91+
config: query.hasConfig() ? deserializeConfig(query.getConfig()) : undefined,
92+
stats: query.hasStats() ? deserializeStats(query.getStats()) : undefined,
9393
};
9494
};
9595

src/utils/serialization/capnpQuerySerializer/capnpQuerySerializer.ts

Lines changed: 7 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -950,12 +950,14 @@ const serializeDimension = (
950950
dimensionCapnp.setNullTileExtent(dimension.nullTileExtent);
951951
dimensionCapnp.setType(dimension.type);
952952

953-
serializeFilterPipeline(
954-
dimensionCapnp.initFilterPipeline(),
955-
dimension.filterPipeline
956-
);
953+
if (dimension.filterPipeline) {
954+
serializeFilterPipeline(
955+
dimensionCapnp.initFilterPipeline(),
956+
dimension.filterPipeline
957+
);
958+
}
957959

958-
if (dimension.domain) {
960+
if (dimension.domain && Object.keys(dimension.domain).length) {
959961
serializeDomainArray(dimensionCapnp.initDomain(), dimension.domain);
960962
}
961963
const { tileExtent } = dimension;

0 commit comments

Comments
 (0)