Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[C#] Bug in line 168 of CArrowArrayStreamExporter.cs #36688

Closed
CurtHagenlocher opened this issue Jul 14, 2023 · 0 comments · Fixed by #36691
Closed

[C#] Bug in line 168 of CArrowArrayStreamExporter.cs #36688

CurtHagenlocher opened this issue Jul 14, 2023 · 0 comments · Fixed by #36691
Assignees
Milestone

Comments

@CurtHagenlocher
Copy link
Contributor

Describe the bug, including details regarding any error messages, version, and platform.

Should be

GCHandle gch = GCHandle.FromIntPtr((IntPtr)(*ptr));

instead of

GCHandle gch = GCHandle.FromIntPtr((IntPtr)ptr);

Component(s)

C#

westonpace pushed a commit that referenced this issue Jul 17, 2023
* Closes: #36688

Authored-by: Curt Hagenlocher <curt@hagenlocher.org>
Signed-off-by: Weston Pace <weston.pace@gmail.com>
@westonpace westonpace added this to the 14.0.0 milestone Jul 17, 2023
chelseajonesr pushed a commit to chelseajonesr/arrow that referenced this issue Jul 20, 2023
* Closes: apache#36688

Authored-by: Curt Hagenlocher <curt@hagenlocher.org>
Signed-off-by: Weston Pace <weston.pace@gmail.com>
raulcd pushed a commit that referenced this issue Jul 25, 2023
* Closes: #36688

Authored-by: Curt Hagenlocher <curt@hagenlocher.org>
Signed-off-by: Weston Pace <weston.pace@gmail.com>
R-JunmingChen pushed a commit to R-JunmingChen/arrow that referenced this issue Aug 20, 2023
* Closes: apache#36688

Authored-by: Curt Hagenlocher <curt@hagenlocher.org>
Signed-off-by: Weston Pace <weston.pace@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants