Skip to content

GuardUsage: restrictions: Use list builder

Ian Jackson requested to merge Diziet/arti:guard-restriction-list into main

Although these do not appear in the config, it seems sensible to get rid of what I think is the remaining ad-hoc list manipulation site.

define_list_config_builder requires that the builder element type be Deserialize. Currently GuardUsageRestriction is a transparent, public enum, so we aren't really exposing anything.

We could introduce GuardUsageRestrictionBuilder now, but since it's not in the config and thereofore only in the public API of the lower crates, we can definitely put that off.

This is part of somewhat related to #447 (closed) and therefore #371 (closed) and #285 (closed)

Found with

git-ls-files | xargs perl -ne 'next unless m/^\#.*derive.*Builder/..m/^\}/; print "$ARGV:$.:$_"' 2>&1 |less +/':.*Vec'
Edited by Ian Jackson

Merge request reports