[sldev] Unit Test Harness: Patch-2

[Gaurav Sharma]

Hi All,

Please find attached the second patch for unit test additions. Patch 2
contains:
- Unit tests for lldatapacker.h, lluuidhashmap.h, v2math.h, v3math.h
- Implementation of safe_snprintf for Windows
- Compilation error fixes related to template use in lluuidhashmap.h
(uncovered by unit tests) and also addition of reset() function

New Issues surfaced:
a. LLUUIDHashMapIter:
   It has a restriction that element removal is not allowed if one or more
iterations are in progress. That is fine, but seems limiting in the sense
that if the iteration does not reach till the end because you want to break
out early, no further removal operations will be allowed until the iterator
goes out of scope or is deleted. To overcome this shortcoming, a reset()
method has been added that can be explicitly called to reset the iterator to
the start (and allow element removal)

b. LLDataPacker (7 out of 14 cases fail)
  pack/unpackFixed and pack/unpackF32 seem to not preserve the float value -
even the approximate_equal check fails.

Next up:
  i) llcommon - llstring.h (more cases), llevent.h
 ii) llmessage - llpartdata.h
iii) llmath - v3dmath.h, m3math.h, xform.h

Thanks,
Gaurav

-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://lists.secondlife.com/pipermail/sldev/attachments/20070302/5f9ec4dc/attachment-0001.htm