Suspicious code in promise.ts

[shicks]

zone.js/lib/common/promise.ts

Line 296 in 9ed5712

promise && (promise = null || resolve(value));

zone.js/lib/common/promise.ts

Line 299 in 9ed5712

promise && (promise = null || reject(error));

I'm rolling out a new lint check and discovered some suspicious code in promise.ts. In the above snippets, null is always falsy, so it is ignored. It can be simplified to simply promise = resolve(value) or promise = reject(value).

[JiaLiPassion]

Thanks, I will fix it!