[issue22120] Fix compiler warnings

2014-08-02 Thread STINNER Victor

STINNER Victor added the comment:

It would be better to only modify clinic for unsigned types, but how do you
check if a type is signed or not?

--

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue22120
___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue22120] Fix compiler warnings

2014-08-02 Thread Serhiy Storchaka

Serhiy Storchaka added the comment:

Either override render() for unsigned type converters, or add new converter 
attribute (in additional to type, cast, conversion_fn, etc).

--

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue22120
___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue22120] Fix compiler warnings

2014-08-02 Thread Charles-François Natali

Charles-François Natali added the comment:

This patch should probably be moved to its own issue.

--

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue22120
___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue22120] Fix compiler warnings

2014-08-02 Thread Charles-François Natali

Changes by Charles-François Natali cf.nat...@gmail.com:


--
Removed message: http://bugs.python.org/msg224550

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue22120
___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue22120] Fix compiler warnings

2014-08-02 Thread STINNER Victor

STINNER Victor added the comment:

Hum, I forgot the attach the most important patch: fix_warnings.patch.

--
Added file: http://bugs.python.org/file36205/fix_warnings.patch

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue22120
___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue22120] Fix compiler warnings

2014-08-01 Thread STINNER Victor

New submission from STINNER Victor:

The issue #22110 enabled more compiler warnings. Attached patch tries to fix 
most of them on Linux.

--
messages: 224528
nosy: haypo
priority: normal
severity: normal
status: open
title: Fix compiler warnings

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue22120
___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue22120] Fix compiler warnings

2014-08-01 Thread STINNER Victor

STINNER Victor added the comment:

clinic.patch: modify clinic.py to generate return_value == (type)-1 instead 
of return_value == -1 to avoid a warning if return_value is unsigned.

--
keywords: +patch
nosy: +larry, neologix
Added file: http://bugs.python.org/file36201/clinic.patch

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue22120
___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue22120] Fix compiler warnings

2014-08-01 Thread Serhiy Storchaka

Serhiy Storchaka added the comment:

I think this should be done only for unsigned integer types. Otherwise it just 
dirty sources and can hide actual bugs.

--
nosy: +serhiy.storchaka

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue22120
___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com