Skip to content

Remove the sort AccountId, don't use -1 as special signal #36

@ehildenb

Description

@ehildenb

AFAIK, the AccountId sort is not needed, instead when we have an undefined account we can just use #undef_account or .Account in the Account sort instead.

Reasoning by extending the builtin sorts will definitely blow up.

Similarly, the newest version uses -1 as a "signal" value for undefined accounts. This will definitely make symbolic reasoning very difficult. It's better to not touch the builtin reasoning sorts (eg. Int or Id) at all.

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions