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

SMem fixes #374

Merged
merged 2 commits into from
Aug 24, 2023
Merged

SMem fixes #374

merged 2 commits into from
Aug 24, 2023
+39 −45

Commits on Aug 23, 2023

  1. harden smem export 

    Harden the logic for exporting smem to prevent segfaults. Previously we had a
segfault with just `smem -x file.soar` because the smem DB is not connected.
Adding an LTI option, however, would connect to the DB first. Change it so that
export (as with backup) requires connecting to the DB first, and gracefully
returns false if not connected.
    @garfieldnate
    garfieldnate committed Aug 23, 2023
    Configuration menu
    Copy the full SHA
    343e367 View commit details
    Browse the repository at this point in the history

  2. connect to SMem DB in init CLI command 

    Previously a simple

smem --init
smem --clear

at the beginning of a source file would fail and prevent the rest of the file
from sourcing. Connect to the DB in the init CLI command so that any following
actions can access the DB. I believe this is the intuitive behavior, and I don't
see anything in the manual that would indicate that this incorrect.
    @garfieldnate
    garfieldnate committed Aug 23, 2023
    Configuration menu
    Copy the full SHA
    2e32249 View commit details
    Browse the repository at this point in the history