SH-3185 Fill in some FIXME/TODO cases

Also added some comments and changed the callback userdata argument
to be an HttpOpRequest rather than a libcurl handle.  Less code,
less clutter.

1 files changed, 6 insertions, 6 deletions

diff --git a/indra/llcorehttp/_httpoperation.cpp b/indra/llcorehttp/_httpoperation.cpp
index d80a8236e6..910dbf1f2f 100644
--- a/indra/llcorehttp/_httpoperation.cpp
+++ b/indra/llcorehttp/_httpoperation.cpp
@@ -91,31 +91,31 @@ void HttpOperation::setReplyPath(HttpReplyQueue * reply_queue,
 
 void HttpOperation::stageFromRequest(HttpService *)
 {
-	// *FIXME:  Message this a better way later.
 	// Default implementation should never be called.  This
 	// indicates an operation making a transition that isn't
 	// defined.
-	llassert_always(false);
+	LL_ERRS("HttpCore") << "Default stateFromRequest method may not be called."
+						<< LL_ENDL;
 }
 
 
 void HttpOperation::stageFromReady(HttpService *)
 {
-	// *FIXME:  Message this a better way later.
 	// Default implementation should never be called.  This
 	// indicates an operation making a transition that isn't
 	// defined.
-	llassert_always(false);
+	LL_ERRS("HttpCore") << "Default stateFromReady method may not be called."
+						<< LL_ENDL;
 }
 
 
 void HttpOperation::stageFromActive(HttpService *)
 {
-	// *FIXME:  Message this a better way later.
 	// Default implementation should never be called.  This
 	// indicates an operation making a transition that isn't
 	// defined.
-	llassert_always(false);
+	LL_ERRS("HttpCore") << "Default stateFromActive method may not be called."
+						<< LL_ENDL;
 }