return 'unimplemented' message rather than panicking

dwrensha · dwrensha · commit 60873834b36e · 2023-01-16T09:02:00.000-05:00
diff --git a/capnp-rpc/src/rpc.rs b/capnp-rpc/src/rpc.rs
@@ -1050,10 +1050,10 @@ impl<VatId> ConnectionState<VatId> {
                                         tmp.borrow_mut().reject(remote_exception_to_error(e?));
                                     }
                                     return_::Canceled(_) => {
-                                        unimplemented!()
+                                        Self::send_unimplemented(&connection_state, &message)?;
                                     }
                                     return_::ResultsSentElsewhere(_) => {
-                                        unimplemented!()
+                                        Self::send_unimplemented(&connection_state, &message)?;
                                     }
                                     return_::TakeFromOtherQuestion(id) => {
                                         if let Some(answer) =
@@ -1080,7 +1080,7 @@ impl<VatId> ConnectionState<VatId> {
                             }
                             None => {
                                 if let return_::TakeFromOtherQuestion(_) = ret.which()? {
-                                    unimplemented!()
+                                    return Self::send_unimplemented(&connection_state, &message);
                                 }
                                 // Looks like this question was canceled earlier, so `Finish`
                                 // was already sent, with `releaseResultCaps` set true so that

Original file line number	Diff line number	Diff line change
`@@ -1050,10 +1050,10 @@ impl<VatId> ConnectionState<VatId> {`
`1050`	`1050`	`tmp.borrow_mut().reject(remote_exception_to_error(e?));`
`1051`	`1051`	`}`
`1052`	`1052`	`return_::Canceled(_) => {`
`1053`		`- unimplemented!()`
	`1053`	`+ Self::send_unimplemented(&connection_state, &message)?;`
`1054`	`1054`	`}`
`1055`	`1055`	`return_::ResultsSentElsewhere(_) => {`
`1056`		`- unimplemented!()`
	`1056`	`+ Self::send_unimplemented(&connection_state, &message)?;`
`1057`	`1057`	`}`
`1058`	`1058`	`return_::TakeFromOtherQuestion(id) => {`
`1059`	`1059`	`if let Some(answer) =`
`@@ -1080,7 +1080,7 @@ impl<VatId> ConnectionState<VatId> {`
`1080`	`1080`	`}`
`1081`	`1081`	`None => {`
`1082`	`1082`	`if let return_::TakeFromOtherQuestion(_) = ret.which()? {`
`1083`		`- unimplemented!()`
	`1083`	`+ return Self::send_unimplemented(&connection_state, &message);`
`1084`	`1084`	`}`
`1085`	`1085`	// Looks like this question was canceled earlier, so `Finish`
`1086`	`1086`	// was already sent, with `releaseResultCaps` set true so that