pcp: Eliminate some redundant pointer validity checking in _ScanArcs().

(Internal change: 2305555)
PixarAnimationStudios · Nov 17, 2023 · 005d75e · 005d75e
1 parent 5416d66
commit 005d75e
Showing 1 changed file with 7 additions and 6 deletions.
diff --git a/pxr/usd/pcp/primIndex.cpp b/pxr/usd/pcp/primIndex.cpp
@@ -664,22 +664,23 @@ _ScanArcs(PcpNodeRef const& node)
     size_t arcs = 0;
     SdfPath const& path = node.GetPath();
     for (SdfLayerRefPtr const& layer: node.GetLayerStack()->GetLayers()) {
-        if (!layer->HasSpec(path)) {
+        SdfLayer const *layerPtr = get_pointer(layer);
+        if (!layerPtr->HasSpec(path)) {
             continue;
         }
-        if (layer->HasField(path, SdfFieldKeys->InheritPaths)) {
+        if (layerPtr->HasField(path, SdfFieldKeys->InheritPaths)) {
             arcs |= _ArcFlagInherits;
         }
-        if (layer->HasField(path, SdfFieldKeys->VariantSetNames)) {
+        if (layerPtr->HasField(path, SdfFieldKeys->VariantSetNames)) {
             arcs |= _ArcFlagVariants;
         }
-        if (layer->HasField(path, SdfFieldKeys->References)) {
+        if (layerPtr->HasField(path, SdfFieldKeys->References)) {
             arcs |= _ArcFlagReferences;
         }
-        if (layer->HasField(path, SdfFieldKeys->Payload)) {
+        if (layerPtr->HasField(path, SdfFieldKeys->Payload)) {
             arcs |= _ArcFlagPayloads;
         }
-        if (layer->HasField(path, SdfFieldKeys->Specializes)) {
+        if (layerPtr->HasField(path, SdfFieldKeys->Specializes)) {
             arcs |= _ArcFlagSpecializes;
         }
     }