Thanks David. This looks very good. I did add one additional error check to raise an error with tostring('c', 'xyz'). If you check for the svn log, I also fixed a lame error message in the old code.
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:
This update contains two semantical changes. First, I noticed that one of the comment lines at the top didn't line up with the others and wanted to fix that in my patch and just forgot about it. Second, the comment for the separator argument still referenced the "S" type and wasn't as clear as I thought it should be.
File Added: array.makestring.2.patch
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:
Logged In: YES
user_id=1125291
Originator: NO
Thanks David. This looks very good. I did add one additional error check to raise an error with tostring('c', 'xyz'). If you check for the svn log, I also fixed a lame error message in the old code.
Logged In: YES
user_id=1011368
Originator: YES
This update contains two semantical changes. First, I noticed that one of the comment lines at the top didn't line up with the others and wanted to fix that in my patch and just forgot about it. Second, the comment for the separator argument still referenced the "S" type and wasn't as clear as I thought it should be.
File Added: array.makestring.2.patch