From e4d27d4b9038733a608bcbda030beff0e95c36df Mon Sep 17 00:00:00 2001
From: Nimrod Gileadi <nimrod@google.com>
Date: Wed, 29 Mar 2023 08:53:23 -0700
Subject: [PATCH] Remove unnecessary null check.

This checked that individual char arrays in the MJCF schema were not null, which would never really happen, and triggered some compiler warnings

PiperOrigin-RevId: 520348916
Change-Id: Ic8b82737460ed3bcf44d9629cd0c7de5782bee8c
---
 src/xml/xml_util.cc | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/xml/xml_util.cc b/src/xml/xml_util.cc
index 32fe7dd3..4c3c5526 100644
--- a/src/xml/xml_util.cc
+++ b/src/xml/xml_util.cc
@@ -133,7 +133,7 @@ mjXSchema::mjXSchema(const char* schema[][mjXATTRNUM], int nrow, bool checkptr)
 
     for (int i=0; i<nrow; i++) {
       // base pointers
-      if (!schema[i] || !schema[i][0]) {
+      if (!schema[i][0]) {
         mju::sprintf_arr(msg, "null pointer found in row %d", i);
         error = msg;
         return;