Jeremy Evans 3e33052fc8 Remove explicit calls to Clover.freeze ...

In production mode and when running the frozen specs, clover_freeze
now calls freeze on all autoloaded constants, and Clover is an
autoloaded constant. Both the specs and config.ru call clover_freeze
unconditionally.

This also removes the override of Clover.freeze.  In production and
when running the frozen specs, it would call
Sequel::Model.freeze_descendants and DB.freeze.  The freezing of all
autoloaded constants handles DB.freeze, and clover_freeze calls
Sequel::Model.freeze_descendants before freezing all autoloaded
constants.

This does remove the
`Sequel::Model.descendants.each(&:finalize_associations)` when running
non-frozen specs, but that's solely an optimization and I don't think
it is worth the complexity.

The `return self if frozen?` code was added to work around a bug
in autoload_hash_branches, where Roda.freeze would break if you
called it the second time on the same object. Now that Clover.freeze
is only called once, it is not needed.

2024-11-14 08:41:56 -08:00

167 B

Raw Permalink Blame History

View Raw