Dear Alexey,

ANV> Error: Abuf is pointer to a BINARY buffer, but ABufSize is the size of STRING
ANV>        buffer.

Fixed :-)

ANV> Also I think it would be good to get a convention to call all STRINGS buffers
ANV> inside API documentatin as "ABuf", but for BINARY buffers as simple "Buf". This
ANV> is a simple "decoration"'s convention, but I think it would be nice.

Well, I prefer not to change parameter names depending on their type.
:-)

ANV> 2. It would be nice to point explicitly the character of the buffer required.
ANV> For example, in TBP_GetName it is explicitly specified that it is _string_
ANV> buffer, and in TBP_GetConfigData it is explicitly cpecified that it is _binary_
ANV> buffer, but in TBP_SetConfigData - nothing specified _explicitly_.

Fixed


ANV> 3. It would be nice to always mention explicitly the measurement's units of
ANV> buffer's size (bytes). For example, in TBPSetConfigData it is mentioned
ANV> explicitly that the size of buffers is "in bytes", but in TBPGetDataProc
ANV> Callback function there is nothing about measurement's units of buffer's size
ANV> mentioned.

Hmmm. Here it says that the size is in bytes :-)  Anyway, all buffer
sizes are in bytes. Exceptions could be made only for UNICODE
functions (in future)


ANV> 4. As I mentioned before, sometimes (more precisely - TBP_Setup and
ANV> TBP_GetConfigData) the AVC_OK is used, but it was never defined explicitly.

Fixed, thank you!




-- 
Read you later
 Stefan                

...Best file compression around:  "DEL *.*" = 100% compression


________________________________________________
http://www.silverstones.com/thebat/TBUDLInfo.html

Reply via email to