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.

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

Jump to bottom

Amount #557

Closed
wants to merge 2 commits into from
Closed

Amount #557

wants to merge 2 commits into from

Commits on Sep 15, 2014

  1. Refactor STAmount: 

    * Remove unused functions
* Mark some observers deprecated
* Clean up declaration parameter names
* Add checked and unchecked constructors
* De-inline unnecessary inlined functions
* Reorder and regroup members into sections
* Move globals from the unity file to the .cpp
* Change some member functions to be free functions
* Put implementation in one .cpp and the test in another .cpp
    @vinniefalco
    vinniefalco committed Sep 15, 2014
    Configuration menu
    Copy the full SHA
    07f613c View commit details
    Browse the repository at this point in the history

  2. Remove unused STAmount constructor and delegate two others 

    No change in functionality.
    @HowardHinnant
    HowardHinnant committed Sep 15, 2014
    Configuration menu
    Copy the full SHA
    38e075d View commit details
    Browse the repository at this point in the history