rework attribute flags

Issue #2358 resolved
Mike Bayer repo owner created an issue

attached is an approach that breaks up PASSIVE_ into bitflags. I tried this using a frozenset() approach instead of integers, but it predictably added a ton of function call overhead.

all tests pass so theoretically this can go right in.

Comments (6)

  1. Former user Account Deleted

    Just FYI: this particular change isn't a priority for me; I think you're the only one who really understands the differences between the different PASSIVE flags. But, for me, this may be blocked by the potential changes to state_session() for being able to load relationships for transient/detached objects, for which I owe you a short use case justification.

    From the patch attached, it looks like you plan to leave the current PASSIVE symbols still defined as they are?

  2. Log in to comment