apr-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From fua...@apache.org
Subject svn commit: r819897 - /apr/apr/trunk/shmem/unix/shm.c
Date Tue, 29 Sep 2009 12:14:46 GMT
Author: fuankg
Date: Tue Sep 29 12:14:45 2009
New Revision: 819897

URL: http://svn.apache.org/viewvc?rev=819897&view=rev
Log:
changed conditionals to avoid a couple of 'statement not reached'
warnings with strict compilers.

Modified:
    apr/apr/trunk/shmem/unix/shm.c

Modified: apr/apr/trunk/shmem/unix/shm.c
URL: http://svn.apache.org/viewvc/apr/apr/trunk/shmem/unix/shm.c?rev=819897&r1=819896&r2=819897&view=diff
==============================================================================
--- apr/apr/trunk/shmem/unix/shm.c (original)
+++ apr/apr/trunk/shmem/unix/shm.c Tue Sep 29 12:14:45 2009
@@ -33,8 +33,7 @@
             return errno;
         }
         return APR_SUCCESS;
-#endif
-#if APR_USE_SHMEM_SHMGET_ANON
+#elif APR_USE_SHMEM_SHMGET_ANON
         if (shmdt(m->base) == -1) {
             return errno;
         }
@@ -56,8 +55,7 @@
         else {
             return apr_file_remove(m->filename, m->pool);
         }
-#endif
-#if APR_USE_SHMEM_MMAP_SHM
+#elif APR_USE_SHMEM_MMAP_SHM
         if (munmap(m->base, m->realsize) == -1) {
             return errno;
         }
@@ -65,8 +63,7 @@
             return errno;
         }
         return APR_SUCCESS;
-#endif
-#if APR_USE_SHMEM_SHMGET
+#elif APR_USE_SHMEM_SHMGET
         /* Indicate that the segment is to be destroyed as soon
          * as all processes have detached. This also disallows any
          * new attachments to the segment. */
@@ -82,10 +79,10 @@
         else {
             return apr_file_remove(m->filename, m->pool);
         }
+#else
+        return APR_ENOTIMPL;
 #endif
     }
-
-    return APR_ENOTIMPL;
 }
 
 APR_DECLARE(apr_status_t) apr_shm_create(apr_shm_t **m,
@@ -172,9 +169,7 @@
         return APR_SUCCESS;
 
 #endif /* APR_USE_SHMEM_MMAP_ZERO */
-#endif /* APR_USE_SHMEM_MMAP_ZERO || APR_USE_SHMEM_MMAP_ANON */
-#if APR_USE_SHMEM_SHMGET_ANON
-
+#elif APR_USE_SHMEM_SHMGET_ANON
         new_m = apr_palloc(pool, sizeof(apr_shm_t));
         new_m->pool = pool;
         new_m->reqsize = reqsize;
@@ -213,9 +208,10 @@
                                   apr_pool_cleanup_null);
         *m = new_m;
         return APR_SUCCESS;
-#endif /* APR_USE_SHMEM_SHMGET_ANON */
+#else
         /* It is an error if they want anonymous memory but we don't have it. */
         return APR_ENOTIMPL; /* requested anonymous but we don't have it */
+#endif
     }
 
     /* Name-based shared memory */
@@ -303,9 +299,7 @@
         *m = new_m;
         return APR_SUCCESS;
 
-#endif /* APR_USE_SHMEM_MMAP_TMP || APR_USE_SHMEM_MMAP_SHM */
-
-#if APR_USE_SHMEM_SHMGET
+#elif APR_USE_SHMEM_SHMGET
         new_m->realsize = reqsize;
 
         /* FIXME: APR_OS_DEFAULT is too permissive, switch to 600 I think. */
@@ -359,10 +353,10 @@
         *m = new_m; 
         return APR_SUCCESS;
 
-#endif /* APR_USE_SHMEM_SHMGET */
+#else
+        return APR_ENOTIMPL;
+#endif
     }
-
-    return APR_ENOTIMPL;
 }
 
 APR_DECLARE(apr_status_t) apr_shm_remove(const char *filename,
@@ -377,14 +371,12 @@
 
 #if APR_USE_SHMEM_MMAP_TMP
     return apr_file_remove(filename, pool);
-#endif
-#if APR_USE_SHMEM_MMAP_SHM
+#elif APR_USE_SHMEM_MMAP_SHM
     if (shm_unlink(filename) == -1) {
         return errno;
     }
     return APR_SUCCESS;
-#endif
-#if APR_USE_SHMEM_SHMGET
+#elif APR_USE_SHMEM_SHMGET
     /* Presume that the file already exists; just open for writing */    
     status = apr_file_open(&file, filename, APR_WRITE,
                            APR_OS_DEFAULT, pool);
@@ -418,10 +410,11 @@
     /* ensure the file has been removed anyway. */
     apr_file_remove(filename, pool);
     return status;
-#endif
+#else
 
     /* No support for anonymous shm */
     return APR_ENOTIMPL;
+#endif
 } 
 
 APR_DECLARE(apr_status_t) apr_shm_destroy(apr_shm_t *m)
@@ -443,16 +436,15 @@
             return errno;
         }
         return APR_SUCCESS;
-#endif /* APR_USE_SHMEM_MMAP_TMP || APR_USE_SHMEM_MMAP_SHM */
-#if APR_USE_SHMEM_SHMGET
+#elif APR_USE_SHMEM_SHMGET
         if (shmdt(m->base) == -1) {
             return errno;
         }
         return APR_SUCCESS;
+#else
+        return APR_ENOTIMPL;
 #endif
     }
-
-    return APR_ENOTIMPL;
 }
 
 APR_DECLARE(apr_status_t) apr_shm_attach(apr_shm_t **m,
@@ -520,8 +512,7 @@
         *m = new_m;
         return APR_SUCCESS;
 
-#endif /* APR_USE_SHMEM_MMAP_TMP || APR_USE_SHMEM_MMAP_SHM */
-#if APR_USE_SHMEM_SHMGET
+#elif APR_USE_SHMEM_SHMGET
         apr_shm_t *new_m;
         apr_status_t status;
         apr_file_t *file;   /* file where metadata is stored */
@@ -566,10 +557,10 @@
         *m = new_m;
         return APR_SUCCESS;
 
-#endif /* APR_USE_SHMEM_SHMGET */
+#else
+        return APR_ENOTIMPL;
+#endif
     }
-
-    return APR_ENOTIMPL;
 }
 
 APR_DECLARE(apr_status_t) apr_shm_detach(apr_shm_t *m)



Mime
View raw message