Skip to content

Commit f4bb152

Browse files
authored
Merge pull request #12941 from CesiumGS/remove-defaultvalue
Remove defaultValue and defaultValue.EMPTY_OBJECT
2 parents 8d67e9e + f1f7a91 commit f4bb152

File tree

3 files changed

+2
-63
lines changed

3 files changed

+2
-63
lines changed

CHANGES.md

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -8,6 +8,8 @@
88

99
#### Breaking Changes :mega:
1010

11+
- The `defaultValue` function has been removed. Instead, use the [nullish coalescing (`??`)](https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Operators/Nullish_coalescing) operator. See the [Coding Guide](https://github.com/CesiumGS/cesium/tree/main/Documentation/Contributors/CodingGuide#default-parameter-values) for usage information and examples.
12+
- `defaultValue.EMPTY_OBJECT` has been removed. Instead, use `Frozen.EMPTY_OBJECT`. See the [Coding Guide](https://github.com/CesiumGS/cesium/tree/main/Documentation/Contributors/CodingGuide#default-parameter-values) for usage information and examples.
1113
- Voxel rendering now requires a WebGL2 context, which is [enabled by default since 1.101](https://github.com/CesiumGS/cesium/pull/10894). For voxel rendering, make sure the `requestWebGl1` flag in `contextOptions` is NOT set to true.
1214

1315
#### Fixes :wrench:

packages/engine/Source/Core/defaultValue.js

Lines changed: 0 additions & 48 deletions
This file was deleted.

packages/engine/Specs/Core/defaultValueSpec.js

Lines changed: 0 additions & 15 deletions
This file was deleted.

0 commit comments

Comments
 (0)