cr comments

Author: stuqdog

src/viam/examples/mlmodel/example_audio_classification_client.cpp

@@ -377,7 +377,7 @@ int main(int argc, char* argv[]) try {
             });
 
             // Print out the top 5 (or fewer) label/score pairs.
-            for (size_t i = 0; i != std::min<std::size_t>(5UL, scored_labels.size()); ++i) {
+            for (size_t i = 0; i != std::min(size_t{5}, scored_labels.size()); ++i) {
                 // TODO: Avoid hardcoding the width here.
                 VIAM_SDK_LOG(info) << boost::format("%1%: %2% %|40t|%3%\n") % i %
                                           *scored_labels[i].label % scored_labels[i].score;

src/viam/sdk/CMakeLists.txt

@@ -287,6 +287,9 @@ else()
   target_sources(viamsdk PRIVATE rpc/private/viam_rust_utils_stubs.cpp)
 endif()
 
+
+# TODO several of these dependencies should properly be attached to `viam_rust_utils`,
+# not `viamsdk`. However, we currently are unable to do so while maintaining compilation
 if (APPLE)
   target_link_libraries(viamsdk PUBLIC "-framework Security")
 elseif (NOT WIN32)