未验证 提交 8ec5276d 编写于 作者: R Ryan Dahl 提交者: GitHub

Add asserts checking that async ops return null (#2463)

上级 d60bdb63
......@@ -91,7 +91,7 @@ export function sendAsync(
data,
false
);
util.assert(response == null);
util.assert(response == null); // null indicates async.
const promise = util.createResolvable<msg.Base>();
promiseTable.set(cmdId, promise);
return promise;
......@@ -106,10 +106,11 @@ export function sendSync(
): null | msg.Base {
const [cmdId, response] = sendInternal(builder, innerType, inner, data, true);
util.assert(cmdId >= 0);
if (response == null || response.length === 0) {
util.assert(response != null); // null indicates async.
if (response!.length === 0) {
return null;
} else {
const bb = new flatbuffers.ByteBuffer(response);
const bb = new flatbuffers.ByteBuffer(response!);
const baseRes = msg.Base.getRootAsBase(bb);
errors.maybeThrowError(baseRes);
return baseRes;
......
Markdown is supported
0% .
You are about to add 0 people to the discussion. Proceed with caution.
先完成此消息的编辑!
想要评论请 注册