Skip to content
This repository has been archived by the owner on Aug 2, 2022. It is now read-only.

Commit

Permalink
Merge pull request #158 from EOSIO/name-overflow-149
Browse files Browse the repository at this point in the history
fix invalid action names #149
  • Loading branch information
bytemaster authored Aug 10, 2017
2 parents ce52012 + 0d661e1 commit 67ef2f4
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 4 deletions.
4 changes: 2 additions & 2 deletions contracts/exchange/exchange.abi
Original file line number Diff line number Diff line change
Expand Up @@ -55,10 +55,10 @@
"action": "sell",
"type": "SellOrder"
},{
"action": "cancel_buy",
"action": "cancelbuy",
"type": "OrderID"
},{
"action": "cancel_sell",
"action": "cancelsell",
"type": "OrderID"
}
],
Expand Down
4 changes: 2 additions & 2 deletions contracts/exchange/exchange.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -307,10 +307,10 @@ extern "C" {
case N(sell):
apply_exchange_sell( currentMessage<exchange::SellOrder>() );
break;
case N(cancel_buy):
case N(cancelbuy):
apply_exchange_cancel_buy( currentMessage<exchange::OrderID>() );
break;
case N(cancel_sell):
case N(cancelsell):
apply_exchange_cancel_sell( currentMessage<exchange::OrderID>() );
break;
default:
Expand Down

1 comment on commit 67ef2f4

@Netherdrake
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What are the constraints on the N macro? Should it only accept lowercase alphabet characters?

Please sign in to comment.