Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

SWIG warnings #96

Closed
olemis opened this issue Dec 29, 2018 · 1 comment
Closed

SWIG warnings #96

olemis opened this issue Dec 29, 2018 · 1 comment
Assignees
Labels
stdev wip Work in progress
Milestone

Comments

@olemis
Copy link
Contributor

olemis commented Dec 29, 2018

Expected Behavior

SWIG generates code silently

Actual Behavior

The following warnings are displayed by SWIG.

swig/pyskycoin_wrap.c swig/pyskycoin.i
swig/include/libskycoin.h:1075: Warning 302: Identifier 'SKY_params_GetDistributionAddresses' redefined (ignored),
swig/python_mem.i:353: Warning 302: previous definition of 'SKY_params_GetDistributionAddresses' (Renamed from 'wrap_SKY_params_GetDistributionAddresses').
swig/include/libskycoin.h:1077: Warning 302: Identifier 'SKY_params_GetUnlockedDistributionAddresses' redefined (ignored),
swig/python_mem.i:360: Warning 302: previous definition of 'SKY_params_GetUnlockedDistributionAddresses' (Renamed from 'wrap_SKY_params_GetUnlockedDistributionAddresses').
swig/include/libskycoin.h:1079: Warning 302: Identifier 'SKY_params_GetLockedDistributionAddresses' redefined (ignored),
swig/python_mem.i:367: Warning 302: previous definition of 'SKY_params_GetLockedDistributionAddresses' (Renamed from 'wrap_SKY_params_GetLockedDistributionAddresses').
swig/include/libskycoin.h:13: Warning 451: Setting a const char * variable may leak memory.
swig/include/libskycoin.h:575: Warning 509: Overloaded method SKY_coin_CreateUnspents(coin__BlockHeader *,Transaction__Handle,coin__UxArray *) effectively ignored,
swig/python_mem.i:114: Warning 509: as it is shadowed by wrap_SKY_coin_CreateUnspents(coin__BlockHeader *,Transaction__Handle,coin_UxOutArray *).
swig/include/libskycoin.h:691: Warning 509: Overloaded method SKY_coin_AddressUxOuts_Keys(AddressUxOuts_Handle,coin__UxArray *) effectively ignored,
swig/python_mem.i:318: Warning 509: as it is shadowed by wrap_SKY_coin_AddressUxOuts_Keys(AddressUxOuts_Handle,cipher_Addresses *).
swig/include/libskycoin.h:693: Warning 509: Overloaded method SKY_coin_AddressUxOuts_Flatten(AddressUxOuts_Handle,coin__UxArray *) effectively ignored,
swig/python_mem.i:275: Warning 509: as it is shadowed by wrap_SKY_coin_AddressUxOuts_Flatten(AddressUxOuts_Handle,coin_UxOutArray *).
swig/include/libskycoin.h:699: Warning 509: Overloaded method SKY_coin_AddressUxOuts_Get(AddressUxOuts_Handle,cipher__Address *,coin__UxArray *) effectively ignored,
swig/python_mem.i:291: Warning 509: as it is shadowed by wrap_SKY_coin_AddressUxOuts_Get(AddressUxOuts_Handle,cipher__Address *,coin_UxOutArray *).
swig/include/libskycoin.h:755: Warning 509: Overloaded method SKY_cipher_GenerateDeterministicKeyPairs(GoSlice,GoInt,coin__UxArray *) effectively ignored,
swig/python_mem.i:47: Warning 509: as it is shadowed by wrap_SKY_cipher_GenerateDeterministicKeyPairs(GoSlice,GoInt,cipher_SecKeys *).
swig/include/libskycoin.h:1029: Warning 509: Overloaded method SKY_coin_Transactions_Hashes(Transactions__Handle,coin__UxArray *) effectively ignored,
swig/python_mem.i:334: Warning 509: as it is shadowed by wrap_SKY_coin_Transactions_Hashes(Transactions__Handle,cipher_SHA256s *).

Steps to Reproduce the Problem

  1. Run make build-swig

Specifications

  • Version: develop
  • Platform: macos

Possible Solution

Unknown

@olemis olemis added stdev wip Work in progress labels Dec 29, 2018
@m4yk3ldev
Copy link
Contributor

@olemis the problem is that you are giving warning because the defaults parameters of libskycoin, change them to the new data types for the GoSlice.

@olemis olemis added this to the 0.25.1 milestone Jan 17, 2019
m4yk3ldev pushed a commit to simelo/pyskycoin.orig that referenced this issue Jan 18, 2019
m4yk3ldev pushed a commit to simelo/pyskycoin.orig that referenced this issue Feb 8, 2019
olemis added a commit that referenced this issue Feb 10, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
stdev wip Work in progress
Projects
None yet
Development

No branches or pull requests

2 participants