fix: updated following review

Author: jerome.fayot

Documentation/Contributors/CodingGuide/README.md

@@ -453,7 +453,7 @@ Cartesian3.fromRadians = function (longitude, latitude, height) {
 this._mapProjection = options.mapProjection ?? new GeographicProjection();
 ```
 
-- If an `options` parameter is optional, use `DefaultValues.EMPTY_OBJECT` or `DefaultValues.EMPTY_ARRAY`, e.g.,
+- If an `options` parameter is optional and never modified afterwards, use `DefaultValues.EMPTY_OBJECT` or `DefaultValues.EMPTY_ARRAY`, this could prevent from unnecessary memory allocations in code critical path, e.g.,
 
 ```javascript
 function BaseLayerPickerViewModel(options) {

packages/engine/Source/Scene/Vector3DTileContent.js

@@ -440,7 +440,7 @@ function initialize(content, arrayBuffer, byteOffset) {
         "POLYGON_COUNT",
         ComponentDatatype.UNSIGNED_INT,
         1,
-      );
+      ); // Workaround for old vector tilesets using the non-plural name;
 
     if (!defined(polygonCounts)) {
       throw new RuntimeError(
@@ -458,7 +458,7 @@ function initialize(content, arrayBuffer, byteOffset) {
         "POLYGON_INDEX_COUNT",
         ComponentDatatype.UNSIGNED_INT,
         1,
-      );
+      ); // Workaround for old vector tilesets using the non-plural name;
 
     if (!defined(polygonIndexCounts)) {
       throw new RuntimeError(
@@ -538,7 +538,7 @@ function initialize(content, arrayBuffer, byteOffset) {
         "POLYLINE_COUNT",
         ComponentDatatype.UNSIGNED_INT,
         1,
-      );
+      ); // Workaround for old vector tilesets using the non-plural name;
 
     if (!defined(polylineCounts)) {
       throw new RuntimeError(