[Bug 217054] [patch] dns/void-zones-tools: update to e38ba1e (adds ARM support)

[bugzilla-noreply at freebsd.org]

https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=217054

cyclaero at gmail.com changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |cyclaero at gmail.com

--- Comment #4 from cyclaero at gmail.com ---
(In reply to mikael.urankar from comment #1)

Hi, I am the developer of the void-zones-tools.

You asked:

 "... char are unsigned on arm, are you sure you need
  'CFLAGS = $(CDEFS) -fsigned-char' on arm?"

Strictly speaking, this not necessary.

However, my development environment is on a x86 system on which char is signed
by default. When I ported my software to arm, I ran into an issue within one
quite old function of thousands, where I lazily assumed that char is always
signed, as you said, it is not on ARM and not on other architectures.

In addition, I got some other projects utilizing the same set of helper
functions, which I want to work the same on every supported system. I revisited
most of my code and it should be agnostic to the signedness of char. Anyway, I
find it important that all the standard C-types are behaving the smae on all
the system that I want to support. So, I would have either added
-funsigned-char for x86 or -fsigned-char for ARM, the latter of which was much
more effortless, and that's why I would like to aak to keep this as it is.

-- 
You are receiving this mail because:
You are the assignee for the bug.