need scriptable string stream implementation (must support nsIInputStream) this is needed to support nsIUploadChannel usage via JS.
i filed a similar bug a while back, but can't seem to find it now. i think this is urgently needed... as in by mozilla0.9.5.
Does this interface handle double-byte strings (like Japanese)? I expected to see wstring. Where do I get the nsIInputStream from?
looks fine: use the /** **/ style comments instead of the c++ // comment. (unless you know more about doxygen that I do, which is probably the case). Also, I am not sure what the mantra is regarding CONTRACTID, CLASSNAME, and CID declarations in an IDL. Rick, Vidur?
brade: you can only upload bytes to a necko channel... you will have to make sure that the data is properly encoded before uploading. dougt: 1- will change to /** **/ comments just to be consistent 2- followed nsILocalFile for an example of how CID, CONTRACTID, and CLASSNAME are declared in XPCOM
This is probably something that we should freeze soon as many scripters will begin using it.
take a look at the bottom of http://www.mozilla.org/projects/embedding/EmbedInterfaceFreeze.html for information about what is needed to freeze a component... -- rick
i'm planning to just implement this interface using the impl's in nsIStringStream.cpp for now... the nsIStringStream.h C++ api will not be touched. we will also probably want a nsIStringOutputStream and perhaps a nsIStringIOStream. but, perhaps those interfaces are less urgently needed.
*** Bug 100171 has been marked as a duplicate of this bug. ***
looks like i'm not going to have time for this now -> 0.9.7
Created attachment 58507 [details] [diff] [review] v1.0 patch this patch takes a slightly different approach.. see nsIStringStream.idl
Comment on attachment 58507 [details] [diff] [review] v1.0 patch actually, this patch still needs work.. the nsIStringInputStream needs a factory.
Created attachment 58776 [details] [diff] [review] v1.1 final patch moving some files around: nsIStringStream.h -> nsIStringStream.idl nsIStringStream.cpp -> nsStringStream.cpp and created: nsStringStream.h
Created attachment 58784 [details] [diff] [review] v1.2 added contributors per "cvs log" to renamed files
Comment on attachment 58784 [details] [diff] [review] v1.2 added contributors per "cvs log" to renamed files got an r=dougt via AIM
Comment on attachment 58784 [details] [diff] [review] v1.2 added contributors per "cvs log" to renamed files firstname.lastname@example.org