Thanks to visit codestin.com
Credit goes to github.com

Skip to content
This repository was archived by the owner on Feb 13, 2025. It is now read-only.

Unused static functions in Stackless/module/channelobject.c #18

Closed
ghost opened this issue Nov 29, 2013 · 2 comments
Closed

Unused static functions in Stackless/module/channelobject.c #18

ghost opened this issue Nov 29, 2013 · 2 comments

Comments

@ghost
Copy link

ghost commented Nov 29, 2013

Originally reported by: RMTEW FULL NAME (Bitbucket: rmtew, GitHub: rmtew)


(originally reported in Trac by @akruis on 2013-04-07 19:21:52)

In preparation for slp 2.7.4 I'm testing various compilers. I'm going to create tickets for warnings.

Version: branch 2.7-slp
Linux amd64
Compiler: clang 3.2
$ ./configure --enable-unicode=ucs4 --prefix=/tmp/slp27 CC=clang CPP='clang -E' CXX='clang++'

clang -pthread -c -fno-strict-aliasing -DSTACKLESS_FRHACK=0 -OPT:Olimit=0 -g -O2 -DNDEBUG -g -fwrapv -O3 -Wall -Wstrict-prototypes  -I. -IInclude -I./Include -I./Stackless   -DPy_BUILD_CORE -o Stackless/module/channelobject.o Stackless/module/channelobject.c
Stackless/module/channelobject.c:76:1: warning: unused function 'slp_channel_has_tasklet' [-Wunused-function]
slp_channel_has_tasklet(PyChannelObject *channel,
^
Stackless/module/channelobject.c:731:32: warning: unused function 'wrap_channel_send_throw' [-Wunused-function]
static CHANNEL_SEND_THROW_HEAD(wrap_channel_send_throw)
                               ^
Stackless/module/channelobject.h:8:13: note: expanded from macro 'CHANNEL_SEND_THROW_HEAD'
        PyObject * func (PyChannelObject *self, PyObject *exc, PyObject *val, PyObject *tb)
                   ^
2 warnings generated.

Kristjan, you contributed these functions. Are they simply dead code?


@ghost
Copy link
Author

ghost commented Nov 29, 2013

Original comment by RMTEW FULL NAME (Bitbucket: rmtew, GitHub: rmtew):


@krisvale on 2013-04-08 11:30:05 said:

One of them is used only in assert, the other is a proper warning that I failed to initialized a struct.
I'll commit a fix asap.

@ghost
Copy link
Author

ghost commented Nov 29, 2013

Original comment by RMTEW FULL NAME (Bitbucket: rmtew, GitHub: rmtew):


@akruis on 2013-04-24 16:18:51 said:

Fix is in changeset [b36e5a180d37].

This issue was closed.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

0 participants