Make the function pointer fields in the C API structs internal.

apache · Jun 10, 2023 · e9365fc · e9365fc
1 parent 154b86c
commit e9365fc
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 6 deletions.
diff --git a/csharp/src/Apache.Arrow/C/CArrowArray.cs b/csharp/src/Apache.Arrow/C/CArrowArray.cs
@@ -38,7 +38,7 @@ public unsafe struct CArrowArray
  public byte** buffers;
  public CArrowArray** children;
  public CArrowArray* dictionary;
- public delegate* unmanaged[Stdcall]<CArrowArray*, void> release;
+ internal delegate* unmanaged[Stdcall]<CArrowArray*, void> release;
  public void* private_data;
 
  /// <summary>

diff --git a/csharp/src/Apache.Arrow/C/CArrowArrayStream.cs b/csharp/src/Apache.Arrow/C/CArrowArrayStream.cs
@@ -35,15 +35,15 @@ public unsafe struct CArrowArrayStream
  ///
  /// Return value: 0 if successful, an `errno`-compatible error code otherwise.
  ///</summary>
- public delegate* unmanaged[Stdcall]<CArrowArrayStream*, CArrowSchema*, int> get_schema;
+ internal delegate* unmanaged[Stdcall]<CArrowArrayStream*, CArrowSchema*, int> get_schema;
 
  /// <summary>
  /// Callback to get the next array. If no error and the array is released, the stream has ended.
  /// If successful, the ArrowArray must be released independently from the stream.
  /// 
  /// Return value: 0 if successful, an `errno`-compatible error code otherwise.
  /// </summary>
- public delegate* unmanaged[Stdcall]<CArrowArrayStream*, CArrowArray*, int> get_next;
+ internal delegate* unmanaged[Stdcall]<CArrowArrayStream*, CArrowArray*, int> get_next;
 
  /// <summary>
  /// Callback to get optional detailed error information. This must only
@@ -54,13 +54,13 @@ public unsafe struct CArrowArrayStream
  /// Return value: pointer to a null-terminated character array describing the last
  /// error, or NULL if no description is available.
  ///</summary>
- public delegate* unmanaged[Stdcall]<CArrowArrayStream*, byte*> get_last_error;
+ internal delegate* unmanaged[Stdcall]<CArrowArrayStream*, byte*> get_last_error;
 
  /// <summary>
  /// Release callback: release the stream's own resources. Note that arrays returned by
  /// get_next must be individually released.
  /// </summary>
- public delegate* unmanaged[Stdcall]<CArrowArrayStream*, void> release;
+ internal delegate* unmanaged[Stdcall]<CArrowArrayStream*, void> release;
 
  public void* private_data;
 

diff --git a/csharp/src/Apache.Arrow/C/CArrowSchema.cs b/csharp/src/Apache.Arrow/C/CArrowSchema.cs
@@ -39,7 +39,7 @@ public unsafe struct CArrowSchema
  public long n_children;
  public CArrowSchema** children;
  public CArrowSchema* dictionary;
- public delegate* unmanaged[Stdcall]<CArrowSchema*, void> release;
+ internal delegate* unmanaged[Stdcall]<CArrowSchema*, void> release;
  public void* private_data;
 
  /// <summary>