test comments

src/people.rs

@@ -68,8 +68,8 @@ type ContextCallback = dyn FnOnce(&mut Context);
 // The purpose of this trait is to allow us to store a vector of different PersonProperties
 // in an object safe way, and use them to emit change events.
 pub trait PersonPropertyHolder {
-    // Given a context, person, and a Vec, a callback is added to that Vec which
-    // can later be called with a new context to emit a change event
+    // Adds a callback to callback_vec which can later be called with a new context
+    // to emit a change event
     fn add_event_callback(
         &self,
         context: &mut Context,
@@ -754,7 +754,8 @@ mod test {
         assert!(*flag.borrow());
     }
 
-    // TODO<[email protected]> This currently doesn't work
+    // TODO([email protected]): Nested derived properties don't currently work; we should
+    // probably either fix this or disallow it.
     #[ignore]
     #[test]
     fn get_derived_property_dependent_on_another_derived() {