Skip to content
This repository has been archived by the owner on Mar 11, 2024. It is now read-only.

Update contract constructors #40

Open
joshma91 opened this issue Oct 27, 2018 · 0 comments
Open

Update contract constructors #40

joshma91 opened this issue Oct 27, 2018 · 0 comments

Comments

@joshma91
Copy link

Solidity has deprecated defining constructors with the same name as the contract. This warning applies to Ownable.sol and Killable.sol.

Warning: Defining constructors as functions with the same name as the contract is deprecated. Use "constructor(...) { ... }" instead.
  function Ownable() {
  ^ (Relevant source part starts here and spans across multiple lines).
,/home/josh/dev/react-auth-box/contracts/zeppelin/ownership/Ownable.sol:13:3: Warning: No visibility specified. Defaulting to "public". 
  function Ownable() {
  ^ (Relevant source part starts here and spans across multiple lines).
,/home/josh/dev/react-auth-box/contracts/zeppelin/ownership/Ownable.sol:22:3: Warning: No visibility specified. Defaulting to "public". 
  function transferOwnership(address newOwner) onlyOwner {
  ^ (Relevant source part starts here and spans across multiple lines).
,/home/josh/dev/react-auth-box/contracts/zeppelin/lifecycle/Killable.sol:12:3: Warning: No visibility specified. Defaulting to "public". 
  function kill() onlyOwner {
  ^ (Relevant source part starts here and spans across multiple lines).
@joshma91 joshma91 changed the title Update Contract Constructors Update contract constructors Oct 27, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant