NumberOfInputs is "Initialize"d to 1. Not much intuitive, I think.
NumberOfInputs is set to 0 on construction (under the present MR). When InitializeFieldList is called, it is "Reset" to 0 (again), vtkDataSetAttributes (argument) is set as the first element, and NumberOfInputs is incremented to 1.
As far as I understand, this should be correct.
NumberOfInputs keeps track of the number of vtkDataSetAttributes already registered. When UnionFieldList is called for the first time, the passed vtkDataSetAttributes is simply added, and the count is set to 1. In the second call to UnionFieldList, -1 may be inserted to Location array for each vtkDataSetAttributes already registered (by PreExtendForUnion at the bottom). In the original implementation, the count was 0 and -1 is not filled, which causes SIGSEGV.
t-ikegami (46606506) at 29 Oct 23:58
Fix SIGSEGV on vtkCompositeDataProbeFilter() (#17690)
t-ikegami (4bca00b0) at 29 Oct 23:48
Fix SIGSEGV on vtkCompositeDataProbeFilter() (#17690)
t-ikegami (83fe6a0d) at 29 Oct 23:43
Merge branch vtk:master into vtkDataSetAttributesFieldList-fixed
... and 42 more commits
t-ikegami at 29 Oct 23:43
t-ikegami (ba4f625c) at 29 Oct 23:42
TestAssignAttribute: fail the test if errors happen
... and 32 more commits
t-ikegami (e3fe2a26) at 29 Oct 23:08
Merge branch vtk:master into vtkDataSetAttributesFieldList-fixed
... and 42 more commits
Mm...I'm not sure if I did it right. Tell me if something wrong.
t-ikegami (f02e2588) at 27 Oct 10:34
Merge branch vtk:master into vtkDataSetAttributesFieldList-fixed
... and 6 more commits
t-ikegami (02797f43) at 26 Oct 23:47
Merge branch vtk:master into vtkDataSetAttributesFieldList-fixed
... and 5 more commits
t-ikegami (063db184) at 26 Oct 23:46
Merge topic 'fix-camera-representation-translucency'
... and 4 more commits
I've created another merge request: !10640 (merged)
Let me know if I should do that within the current merge request.
t-ikegami (8b7fc6b7) at 26 Oct 23:08
Merge branch vtk:master into vtkDataSetAttributesFieldList-fixed
... and 6916 more commits
I'll try...within a week.
I hope so. Paraview 5.11.2 is still affected by #17690.