Skip to content

Commit

Permalink
remove unneded flag setting, fix printf
Browse files Browse the repository at this point in the history
  • Loading branch information
paleolimbot committed Jan 15, 2024
1 parent 7e46f47 commit 3da7a6f
Show file tree
Hide file tree
Showing 2 changed files with 1 addition and 7 deletions.
6 changes: 0 additions & 6 deletions python/setup.py
Original file line number Diff line number Diff line change
Expand Up @@ -66,12 +66,6 @@ def get_version(pkg_path):
extra_link_args = []
extra_define_macros = []

# Setting the CFLAGS environment variable doesn't seem sufficient in all cases to add
# extra flags (e.g., -std=c99 on very old gcc).
more_compile_args = os.getenv("NANOARROW_PYTHON_CFLAGS")
if more_compile_args:
extra_compile_args += re.split(r"\s+", more_compile_args)

setup(
ext_modules=[
Extension(
Expand Down
2 changes: 1 addition & 1 deletion python/src/nanoarrow/_lib.pyx
Original file line number Diff line number Diff line change
Expand Up @@ -915,7 +915,7 @@ cdef class CBufferView:
if format_const != NULL:
snprintf(self._format, sizeof(self._format), "%s", format_const)
else:
snprintf(self._format, sizeof(self._format), "%ds", <int>self._element_size_bits // 8)
snprintf(self._format, sizeof(self._format), "%ds", <int>(self._element_size_bits // 8))

def __getbuffer__(self, Py_buffer *buffer, int flags):
if self._device.device_type != ARROW_DEVICE_CPU:
Expand Down

0 comments on commit 3da7a6f

Please sign in to comment.