Disable triangle rendering when triangleIndex is -1 to avoid Null Pointer for Convex MeshCollider

Author: 3DI70R

Assets/Scripts/Runtime/PhysicsSensorUtils.cs

@@ -82,8 +82,13 @@ public static void DrawNormal(RaycastHit hit)
         public static void HighlightMeshVertices(RaycastHit hit)
         {
             var meshCollider = hit.collider as MeshCollider;
+            
             if (meshCollider != null)
             {
+                // Convex
+                if(hit.triangleIndex == -1)
+                    return;
+                
                 var mesh = meshCollider.sharedMesh;
 
                 if (mesh != lastInspectedMesh)

README-ru.md

@@ -102,4 +102,6 @@ Unity **2018.1** или выше
     - Улучшение: Форматирование для документации, Всплывающие окошки в с документацией у полей
     - Улучшение: Обновление стиля окошка с информацией о попадании, также теперь он не обрезается о границу окна
     - Улучшение: Кэширование вершин для отображения треугольников
-    - Улучшение: "Lazy Allocation" теперь только для чтения, нельзя анимировать и можно проставить только из инспектора, так как его значение используется только при инициализации компонента
+    - Улучшение: "Lazy Allocation" теперь только для чтения, нельзя анимировать и можно проставить только из инспектора, так как его значение используется только при инициализации компонента
+* 1.1.4
+    - Багфикс: Out of bounds exception при попытке отрисовать треугольник на Convex MeshCollider (при таком случае, triangleIndex == -1)

README.md

@@ -119,4 +119,6 @@ Otherwise, you can just copy files from `Assets/Scripts/Runtime` to your project
     - Improvement: Documentation formatting fixes, Tooltips for editor
     - Improvement: Hit information window style update, also now it doesn't clip out of view
     - Improvement: Vertex caching for hit information overlay
-    - Improvement: "Lazy Allocation" now readonly, non keyable and can be set only from inspector, because it is used only during component initialization
+    - Improvement: "Lazy Allocation" now readonly, non keyable and can be set only from inspector, because it is used only during component initialization
+* 1.1.4
+    - Bugfix: Out of bounds exception when triangle highlight gizmo tries to display triangle on convex mesh (triangleIndex is -1 in this case)

package.json

@@ -1,7 +1,7 @@
 {
     "name": "ru.threedisevenzeror.sensorkit",
     "displayName": "Sensor Kit",
-    "version": "1.1.3",
+    "version": "1.1.4",
     "unity": "2018.1",
     "description": "Collection of wrapped Physics.* methods with ability to place casts in scene and configure parameters without hardcoding values",
     "keywords": ["Physics", "Raycast", "Utility"],