Skip to content

Shareable: improved code to styling conventions, added functionality todo with owners #208

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.

Sign up for GitHub

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

Closed
RFV wants to merge 4 commits into from
Closed

Shareable: improved code to styling conventions, added functionality todo with owners #208

Changes from 1 commit
Commits
Show all changes
4 commits
Select commit Hold shift + click to select a range
193302c
Fixed modifier naming styles
May 3, 2017
18ffade
fixed one function parameter naming style
May 3, 2017
ab9c48e
added ownerAdd, ownerRemove, changeRequirement, and thus removed the …
May 3, 2017
1b0f2ad
Changed the Zeppelin Destroy functionality fundamentally, to limit lo…
May 4, 2017
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
fixed one function parameter naming style
  • Loading branch information
Riaan F Venter committed May 3, 2017
commit 18ffade3d7ecdfb3c5d858288e22f4b87928c50c
4 changes: 2 additions & 2 deletions contracts/ownership/Shareable.sol
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -86,8 +86,8 @@ contract Shareable {
}

// Gets an owner by 0-indexed position (using numOwners as the count)
function getOwner(uint ownerIndex) external constant returns (address) {
return address(owners[ownerIndex + 1]);
function getOwner(uint _ownerIndex) external constant returns (address) {
return address(owners[_ownerIndex + 1]);
}

function isOwner(address _addr) constant returns (bool) {
Expand Down