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

Remove AList from use in orders/order lists #222

Merged
merged 2 commits into from
Oct 7, 2024
Merged

Commits on Sep 30, 2024

  1. Another rename on MarketType

    Actually do a named enum correctly in order to enforce typechecking.
    Also, modified editor settings to get rid of dumb extra spaces, and
    wrap things better, so these will occur over time as I edit files.
    jt-traub committed Sep 30, 2024
    Configuration menu
    Copy the full SHA
    8574744 View commit details
    Browse the repository at this point in the history

Commits on Oct 7, 2024

  1. Remove AList usage from all orders

    This modifies all orders to no longer be ALists.  Also moves
    them to be shared_ptr rather than using raw pointers with
    new/delete.
    
    In most cases, I chose to use std::vector for the list of orders
    but in some (two) cases I chose std::list since there was a need
    to insert at the head.  Choosing vector over list is preferred in
    general as it has better speed, but it makes inserting at the head
    harder.
    jt-traub committed Oct 7, 2024
    Configuration menu
    Copy the full SHA
    51f9b49 View commit details
    Browse the repository at this point in the history