Tidied up buildSELECT() a bit:

Tidied up buildSELECT() a bit:

1. Sprinkled the const qualifier here and there.

2. Moved some variable declarations to get them closer to the point of
first use, and to limit their scope.

3. Renamed some variables to better reflect their meaning.

4. Split a couple of variables into multiple variables, instead of using
them for multiple unrelated purposes.

5. Plugged a memory leak in the case of an error return.

6. Added comments, including a Doxygen-style comment at the top of the
function.

M    Open-ILS/src/c-apps/oils_sql.c

git-svn-id: svn://svn.open-ils.org/ILS/trunk@18240 dcc99617-32d9-48b4-a31d-7c20da2025e4