aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorBoris Kolpackov <boris@codesynthesis.com>2011-11-30 12:13:33 +0200
committerBoris Kolpackov <boris@codesynthesis.com>2011-11-30 12:13:33 +0200
commita262bf3d1102a413d2c8161e469ab6f3df8baf96 (patch)
tree3ce794a292c112ab5bed235e3f784f6571530754
parent24c8951999e1dc9dc4419dcf5415576ae8b0d55d (diff)
Cosmetic changes
-rw-r--r--odb/oracle/error.cxx62
-rw-r--r--odb/oracle/error.hxx6
2 files changed, 37 insertions, 31 deletions
diff --git a/odb/oracle/error.cxx b/odb/oracle/error.cxx
index b822256..f52de31 100644
--- a/odb/oracle/error.cxx
+++ b/odb/oracle/error.cxx
@@ -20,38 +20,38 @@ namespace odb
namespace oracle
{
static void
- translate_error (void* h, ub4 t, sword s, connection* conn)
+ translate_error (void* h, ub4 htype, sword r, connection* conn)
{
- assert (s != OCI_SUCCESS && s != OCI_SUCCESS_WITH_INFO);
+ assert (r != OCI_SUCCESS && r != OCI_SUCCESS_WITH_INFO);
- if (s != OCI_ERROR)
+ switch (r)
{
- switch (s)
+ case OCI_STILL_EXECUTING:
{
- case OCI_STILL_EXECUTING:
- {
- throw database_exception (0, "statement still executing");
- break;
- }
- case OCI_NEED_DATA:
- case OCI_NO_DATA:
- {
- throw database_exception (0, "unhandled OCI_*_DATA condition");
- break;
- }
- case OCI_INVALID_HANDLE:
- {
- throw invalid_oci_handle ();
- break;
+ throw database_exception (0, "statement still executing");
+ break;
}
+ case OCI_NEED_DATA:
+ case OCI_NO_DATA:
+ {
+ throw database_exception (0, "unhandled OCI_*_DATA condition");
+ break;
+ }
+ case OCI_INVALID_HANDLE:
+ {
+ throw invalid_oci_handle ();
+ break;
+ }
+ default:
+ {
+ break;
}
}
- sword r;
sb4 e;
char b[512]; // Error message will be truncated if it does not fit.
- if (t == OCI_HTYPE_ERROR)
+ if (htype == OCI_HTYPE_ERROR)
{
// Mark the connection as failed if necessary.
//
@@ -104,7 +104,13 @@ namespace odb
for (sb4 i (1);; ++i)
{
- r = OCIErrorGet (h, i, 0, &e, reinterpret_cast<OraText*> (b), 512, t);
+ r = OCIErrorGet (h,
+ i,
+ 0,
+ &e,
+ reinterpret_cast<OraText*> (b),
+ 512,
+ htype);
if (r == OCI_NO_DATA)
break;
@@ -116,7 +122,7 @@ namespace odb
nc = code_deadlock;
else if (e == 51 || // Timeout occurred while waiting for a resource.
e == 54 || // Resource busy and acquisition timeout expired.
- e == 2049) // Distributed lock timeout.
+ e == 2049) // Distributed lock timeout.
nc = code_timeout;
else if (e == 28 || // Session has been killed.
e == 3113 || // End-of-file on communication channel.
@@ -166,7 +172,7 @@ namespace odb
&e,
reinterpret_cast<OraText*> (b),
512,
- t);
+ htype);
if (r == OCI_NO_DATA)
break;
@@ -178,15 +184,15 @@ namespace odb
}
void
- translate_error (OCIError* h, sword result)
+ translate_error (OCIError* h, sword r)
{
- translate_error (h, OCI_HTYPE_ERROR, result, 0);
+ translate_error (h, OCI_HTYPE_ERROR, r, 0);
}
void
- translate_error (connection& c, sword result)
+ translate_error (connection& c, sword r)
{
- translate_error (c.error_handle (), OCI_HTYPE_ERROR, result, &c);
+ translate_error (c.error_handle (), OCI_HTYPE_ERROR, r, &c);
}
void
diff --git a/odb/oracle/error.hxx b/odb/oracle/error.hxx
index a715bce..62639db 100644
--- a/odb/oracle/error.hxx
+++ b/odb/oracle/error.hxx
@@ -21,16 +21,16 @@ namespace odb
// exception.
//
LIBODB_ORACLE_EXPORT void
- translate_error (OCIError* h, sword result);
+ translate_error (OCIError*, sword result);
LIBODB_ORACLE_EXPORT void
- translate_error (connection& c, sword result);
+ translate_error (connection&, sword result);
// Translate an OCI error given an environment handle error and throw
// an appropriate exception.
//
LIBODB_ORACLE_EXPORT void
- translate_error (OCIEnv* h);
+ translate_error (OCIEnv*);
}
}