audioop.c giving signed/unsigned warnings on Windows

[tjguk]

BPO	19418
Nosy	@tim-one, @vstinner, @tjguk, @serhiy-storchaka
PRs	[2.7] bpo-19418: audioop.c: Fix warnings on -0x80000000 #7453
Files	issue19418.2.diff

^{Note: these values reflect the state of the issue at the time it was migrated and might not reflect the current state.}

Show more details

GitHub fields:

assignee = None
closed_at = <Date 2013-10-31.17:51:52.930>
created_at = <Date 2013-10-27.19:44:37.950>
labels = ['build']
title = 'audioop.c giving signed/unsigned warnings on Windows'
updated_at = <Date 2018-06-06.15:51:10.003>
user = 'https://github.com/tjguk'

bugs.python.org fields:

activity = <Date 2018-06-06.15:51:10.003>
actor = 'vstinner'
assignee = 'none'
closed = True
closed_date = <Date 2013-10-31.17:51:52.930>
closer = 'tim.golden'
components = ['Build']
creation = <Date 2013-10-27.19:44:37.950>
creator = 'tim.golden'
dependencies = []
files = ['32401']
hgrepos = []
issue_num = 19418
keywords = ['patch']
message_count = 17.0
messages = ['201485', '201486', '201490', '201491', '201492', '201495', '201496', '201499', '201501', '201575', '201576', '201579', '201584', '201798', '201833', '201835', '318836']
nosy_count = 5.0
nosy_names = ['tim.peters', 'vstinner', 'tim.golden', 'python-dev', 'serhiy.storchaka']
pr_nums = ['7453']
priority = 'normal'
resolution = 'fixed'
stage = 'resolved'
status = 'closed'
superseder = None
type = 'compile error'
url = 'https://bugs.python.org/issue19418'
versions = ['Python 3.3', 'Python 3.4']

[tjguk]

Modules/audioop.c is giving compile warnings on Windows against lines 18 & 437.

..\Modules\audioop.c(18): warning C4146: unary minus operator applied to unsigned type, result still unsigned
..\Modules\audioop.c(437): warning C4146: unary minus operator applied to unsigned type, result still unsigned

[tjguk]

The attached patch appears to fix the problem.

[serhiy-storchaka]

But it adds warnings on 32-bit platform with GCC.

/home/serhiy/py/cpython/Modules/audioop.c:18:61: warning: integer overflow in expression [-Woverflow]
/home/serhiy/py/cpython/Modules/audioop.c: In function ‘audioop_minmax’:
/home/serhiy/py/cpython/Modules/audioop.c:437:33: warning: integer overflow in expression [-Woverflow]

[serhiy-storchaka]

What if use (int)-0x80000000 instead of -(int)0x80000000? Does it silence warnings on Windows?

[tjguk]

I don't think it will, given MS description of the compiler warning:

http://msdn.microsoft.com/en-us/library/4kh09110%28v=vs.100%29.aspx

but I'll certainly give it a go.

[tim-one]

Use

-0x7FFFFFFF-1

Nobody should complain about that, because it's standard C ;-)

On Sun, Oct 27, 2013 at 3:20 PM, Tim Golden <report@bugs.python.org> wrote:

Tim Golden added the comment:

I don't think it will, given MS description of the compiler warning:

http://msdn.microsoft.com/en-us/library/4kh09110%28v=vs.100%29.aspx

but I'll certainly give it a go.

----------

---

Python tracker <report@bugs.python.org>
<http://bugs.python.org/issue19418\>

---

---

Python-bugs-list mailing list
Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/tim.peters%40gmail.com

[serhiy-storchaka]

LGTM. Do you want commit a patch?

[tim-one]

@serhiy, nope, pressed for time today :-(

[tjguk]

Good idea. I'll test against Win32/64 & Linux 32

[tjguk]

The attached patch (using Tim Peters' -1 trick) builds & tests ok on Win32/64 & Linux 32.

[vstinner]

You should add a comment to explain why the trick is required (because of compilers not supporting -0x80000000 literal without a warning).

[tjguk]

Good point. Updated patch.

[serhiy-storchaka]

LGTM.