Changeset 5544:16ca0566b0e9 in roaraudio for libroar/vio_proto.c

Timestamp:

06/17/12 10:36:37 (12 years ago)

Author:

phi

Branch:

default

Phase:

public

Message:

Improved error handling (including on win32) (Closes: #235)

File:

• libroar/vio_proto.c (modified) (2 diffs)

libroar/vio_proto.c

@@ -502 +502 @@
  int  oeflen = 4;
  int  aligned = 1;
+ int  error;
 
  ROAR_DBG("roar_vio_open_proto_http(calls=%p, dst=%p, host='%s', file='%s') = ?", calls, dst, host, file);
@@ -563 +564 @@
   ROAR_DBG("roar_vio_open_proto_http(*) = -1 // status=%i", status);
   roar_buffer_free(bufbuf);
-  switch (status) {
-   // 3xx-Redirects:
-   case 301: roar_err_set(ROAR_ERROR_MOVEDPERM); break;
-   case 303: roar_err_set(ROAR_ERROR_SEEOTHER); break;
-   case 305: roar_err_set(ROAR_ERROR_USEPROXY); break;
-   case 307: roar_err_set(ROAR_ERROR_MOVEDTEMP); break;
-   // 4xx-Client errors:
-   case 400: roar_err_set(ROAR_ERROR_INVAL); break;
-   case 403: roar_err_set(ROAR_ERROR_PERM); break;
-   case 404: roar_err_set(ROAR_ERROR_NOENT); break;
-   case 408: roar_err_set(ROAR_ERROR_TIMEDOUT); break;
-   case 410: roar_err_set(ROAR_ERROR_GONE); break;
-   case 423: roar_err_set(ROAR_ERROR_BUSY); break;
-   case 426: roar_err_set(ROAR_ERROR_SWITCHPROTO); break;
-   // 5xx-Server errors:
-   case 501: roar_err_set(ROAR_ERROR_NOSYS); break;
-   case 504: roar_err_set(ROAR_ERROR_TIMEDOUT); break;
-   case 505: roar_err_set(ROAR_ERROR_NSVERSION); break;
-   case 507: roar_err_set(ROAR_ERROR_NOSPC); break;
-   // default case:
-   default:
-     roar_err_set(ROAR_ERROR_UNKNOWN);
-    break;
+  if ( roar_err_convert(&error, ROAR_ERROR_TYPE_ROARAUDIO, status, ROAR_ERROR_TYPE_HTTP) != ROAR_ERROR_NONE ) {
+   roar_err_set(ROAR_ERROR_UNKNOWN);
+  } else {
+   roar_err_set(error);
   }
   return -1;