fix(3p/nix/libexpr): Remove default & copy constructors for VectorBindings r/1288

Paired-With: Luke Granger-Brown <git@lukegb.com>
Paired-With: Vincent Ambo <mail@tazj.in>
Paired-With: Perry Lorier <isomer@tvl.fyi>

Change-Id: Ief2e59d461452ce599abc63f6ebcfa07a7062491
Reviewed-on: https://cl.tvl.fyi/c/depot/+/1161
Tested-by: BuildkiteCI
Reviewed-by: lukegb <lukegb@tvl.fyi>

1 files changed, 2 insertions, 1 deletions

diff --git a/third_party/nix/src/libexpr/attr-set.cc b/third_party/nix/src/libexpr/attr-set.cc
index 5ef63a129cdc..31b5bdcef3b1 100644
--- a/third_party/nix/src/libexpr/attr-set.cc
+++ b/third_party/nix/src/libexpr/attr-set.cc
@@ -131,7 +131,6 @@ void BTreeBindings::merge(Bindings& other) {
 
 class VectorBindings : public Bindings {
  public:
-  VectorBindings() {};
   VectorBindings(size_t capacity) : attributes_() {
     attributes_.reserve(capacity);
   };
@@ -145,6 +144,8 @@ class VectorBindings : public Bindings {
   void merge(Bindings& other) override;
   [[deprecated]] virtual std::vector<const Attr*> lexicographicOrder() override;
 
+  VectorBindings(VectorBindings&) = delete;
+
  private:
   AttributeVector attributes_;
 };