[SymForce] return size_t instead of int32_t from values::NumEntries()

Topic: chaoqu_symforce_values_num_entries Reviewers: aaron, brad, nathan, hayk GitOrigin-RevId: 527716e9b97b3fe89d0db8362686e46304c58b6b
symforce-org · Mar 8, 2023 · 35fd2d9 · 35fd2d9
1 parent 951884c
commit 35fd2d9
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/symforce/opt/values.cc b/symforce/opt/values.cc
@@ -65,7 +65,7 @@ void Values<Scalar>::UpdateOrSet(const index_t& index, const Values<Scalar>& oth
 }
 
 template <typename Scalar>
-int32_t Values<Scalar>::NumEntries() const {
+size_t Values<Scalar>::NumEntries() const {
   return map_.size();
 }
 

diff --git a/symforce/opt/values.h b/symforce/opt/values.h
@@ -93,7 +93,7 @@ class Values {
   /**
    * Number of keys.
    */
-  int32_t NumEntries() const;
+  size_t NumEntries() const;
 
   /**
    * Has zero keys.