Changeset 13461

Timestamp:

2006-12-08T12:06:47+13:00 (17 years ago)

Author:

kjdon

Message:

added some comments for decode_cgi_arg method

File:

• trunk/gsdl/src/recpt/cgiutils.cpp (modified) (2 diffs)

trunk/gsdl/src/recpt/cgiutils.cpp

@@ -270 +270 @@
 // convert %xx and + to their appropriate equivalents
 // IE 6.0 and later use "%u" followed by 4 hex digits... MS IIS extension!
+// NOTE: this method is crap. It assumes the input encoding is utf-8. If it 
+// actually was, then this returns utf-8, and needs to_uni on the 
+// result to get it back to unicode. If the encoding wasn't utf-8, then the 
+// output may be crap. Seems to work for 8 bit encodings.
+// Really, this should be given the encoding, and should always return unicode. 
 void decode_cgi_arg (text_t &argstr) {
   text_t::iterator in = argstr.begin();
@@ -320 +325 @@
   // remove the excess characters
   argstr.erase (out, end);
+  
 }