Repository navigation
PyMethodDef ml_name is char* instead of const char* #65218
Description
| BPO | 21019 |
|---|---|
| Nosy | @loewis, @methane, @hvenev |
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 2019-03-28.13:20:01.824>
created_at = <Date 2014-03-22.10:14:14.015>
labels = ['extension-modules']
title = 'PyMethodDef ml_name is char* instead of const char*'
updated_at = <Date 2019-03-28.13:20:01.818>
user = 'https://github.com/hvenev'bugs.python.org fields:
activity = <Date 2019-03-28.13:20:01.818>
actor = 'methane'
assignee = 'none'
closed = True
closed_date = <Date 2019-03-28.13:20:01.824>
closer = 'methane'
components = ['Extension Modules']
creation = <Date 2014-03-22.10:14:14.015>
creator = 'h.venev'
dependencies = []
files = []
hgrepos = []
issue_num = 21019
keywords = []
message_count = 6.0
messages = ['214451', '214463', '214474', '214503', '214506', '339033']
nosy_count = 5.0
nosy_names = ['loewis', 'Arfrever', 'methane', 'tshepang', 'h.venev']
pr_nums = []
priority = 'normal'
resolution = 'fixed'
stage = 'resolved'
status = 'closed'
superseder = None
type = None
url = 'https://bugs.python.org/issue21019'
versions = ['Python 3.4']
Activity
hvenev commentedon Mar 22, 2014
It would be better if string literals could be used there.
loewis commentedon Mar 22, 2014
Why are you suggesting that string literals cannot be used? Works fine for me.
hvenev commentedon Mar 22, 2014
error: deprecated conversion from string constant to ‘char*’
I like -Werror.
loewis commentedon Mar 22, 2014
I still can't reproduce the problem, not even with -Werror:
loewis@dinsdale:
$ gcc -Wall -Werror -o a a.c$ gcc --versionloewis@dinsdale:
gcc (Debian 4.7.2-5) 4.7.2
Copyright (C) 2012 Free Software Foundation, Inc.
This is free software; see the source for copying conditions. There is NO
warranty; not even for MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.
loewis@dinsdale:~$
hvenev commentedon Mar 22, 2014
g++
methane commentedon Mar 28, 2019
It is const char* now.