From a6f4202ac5527569f5cef062b5fdd636ff0aeaca Mon Sep 17 00:00:00 2001
From: Robbert van der Helm <mail@robbertvanderhelm.nl>
Date: Sun, 20 Mar 2022 02:44:13 +0100
Subject: [PATCH] Add a todo for combining the Params methods

---
 src/param/internals.rs | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/src/param/internals.rs b/src/param/internals.rs
index f5f80fe4..f91e23a2 100644
--- a/src/param/internals.rs
+++ b/src/param/internals.rs
@@ -43,6 +43,9 @@ pub trait Params {
     // NOTE: These types use `String` even though for the `Params` derive macro `&'static str` would
     //       have been fine to be able to support custom reusable Params implemnetations.
 
+    // TODO: Combine `param_map`, `param_groups`, and `param_ids` into a single function that
+    //       returns a vector of tuples
+
     /// Create a mapping from unique parameter IDs to parameters. This is done for every parameter
     /// field marked with `#[id = "stable_name"]`. Dereferencing the pointers stored in the values
     /// is only valid as long as this pinned object is valid.