You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@lucy.apache.org by ma...@apache.org on 2011/06/19 20:40:09 UTC
[lucy-commits] svn commit: r1137407 - in /incubator/lucy/branches/0.1: ./
charmonizer/src/Charmonizer/ charmonizer/src/Charmonizer/Core/
charmonizer/src/Charmonizer/Probe/ charmonizer/src/Charmonizer/Test/
Author: marvin
Date: Sun Jun 19 18:40:08 2011
New Revision: 1137407
URL: http://svn.apache.org/viewvc?rev=1137407&view=rev
Log:
LUCY-156 Const-ify static char array declarations.
Apply "const" qualifier to numerous local char* and char[] declarations.
Modified:
incubator/lucy/branches/0.1/ (props changed)
incubator/lucy/branches/0.1/charmonizer/src/Charmonizer/Core/Compiler.c
incubator/lucy/branches/0.1/charmonizer/src/Charmonizer/Core/Dir.c
incubator/lucy/branches/0.1/charmonizer/src/Charmonizer/Core/HeaderChecker.c
incubator/lucy/branches/0.1/charmonizer/src/Charmonizer/Core/OperatingSystem.c
incubator/lucy/branches/0.1/charmonizer/src/Charmonizer/Core/Stat.c
incubator/lucy/branches/0.1/charmonizer/src/Charmonizer/Probe.c
incubator/lucy/branches/0.1/charmonizer/src/Charmonizer/Probe/AtomicOps.c
incubator/lucy/branches/0.1/charmonizer/src/Charmonizer/Probe/FuncMacro.c
incubator/lucy/branches/0.1/charmonizer/src/Charmonizer/Probe/Headers.c
incubator/lucy/branches/0.1/charmonizer/src/Charmonizer/Probe/Integers.c
incubator/lucy/branches/0.1/charmonizer/src/Charmonizer/Probe/LargeFiles.c
incubator/lucy/branches/0.1/charmonizer/src/Charmonizer/Probe/Memory.c
incubator/lucy/branches/0.1/charmonizer/src/Charmonizer/Probe/VariadicMacros.c
incubator/lucy/branches/0.1/charmonizer/src/Charmonizer/Test/TestFuncMacro.c
Propchange: incubator/lucy/branches/0.1/
------------------------------------------------------------------------------
--- svn:mergeinfo (original)
+++ svn:mergeinfo Sun Jun 19 18:40:08 2011
@@ -1 +1 @@
-/incubator/lucy/trunk:1134011,1134355,1134472,1134843-1134844
+/incubator/lucy/trunk:1134011,1134355,1134472,1134843-1134845
Modified: incubator/lucy/branches/0.1/charmonizer/src/Charmonizer/Core/Compiler.c
URL: http://svn.apache.org/viewvc/incubator/lucy/branches/0.1/charmonizer/src/Charmonizer/Core/Compiler.c?rev=1137407&r1=1137406&r2=1137407&view=diff
==============================================================================
--- incubator/lucy/branches/0.1/charmonizer/src/Charmonizer/Core/Compiler.c (original)
+++ incubator/lucy/branches/0.1/charmonizer/src/Charmonizer/Core/Compiler.c Sun Jun 19 18:40:08 2011
@@ -37,17 +37,17 @@ static char *try_app_name = NULL;
/* Detect a supported compiler, or assume a generic GCC-compatible compiler
* and hope for the best. */
#ifdef __GNUC__
-static char *include_flag = "-I ";
-static char *object_flag = "-o ";
-static char *exe_flag = "-o ";
+static const char *include_flag = "-I ";
+static const char *object_flag = "-o ";
+static const char *exe_flag = "-o ";
#elif defined(_MSC_VER)
-static char *include_flag = "/I";
-static char *object_flag = "/Fo";
-static char *exe_flag = "/Fe";
+static const char *include_flag = "/I";
+static const char *object_flag = "/Fo";
+static const char *exe_flag = "/Fe";
#else
-static char *include_flag = "-I ";
-static char *object_flag = "-o ";
-static char *exe_flag = "-o ";
+static const char *include_flag = "-I ";
+static const char *object_flag = "-o ";
+static const char *exe_flag = "-o ";
#endif
/* Clean up the files associated with CC_capture_output().
@@ -264,7 +264,7 @@ S_clean_up_try(void) {
static void
S_do_test_compile(void) {
- char *code = "int main() { return 0; }\n";
+ const char *code = "int main() { return 0; }\n";
chaz_bool_t success;
if (Util_verbosity) {
Modified: incubator/lucy/branches/0.1/charmonizer/src/Charmonizer/Core/Dir.c
URL: http://svn.apache.org/viewvc/incubator/lucy/branches/0.1/charmonizer/src/Charmonizer/Core/Dir.c?rev=1137407&r1=1137406&r2=1137407&view=diff
==============================================================================
--- incubator/lucy/branches/0.1/charmonizer/src/Charmonizer/Core/Dir.c (original)
+++ incubator/lucy/branches/0.1/charmonizer/src/Charmonizer/Core/Dir.c Sun Jun 19 18:40:08 2011
@@ -35,7 +35,7 @@ static char mkdir_command[7];
char *Dir_mkdir_command = mkdir_command;
/* Source code for standard POSIX mkdir */
-static char posix_mkdir_code[] =
+static const char posix_mkdir_code[] =
QUOTE( #include <%s> )
QUOTE( int main(int argc, char **argv) { )
QUOTE( if (argc != 2) { return 1; } )
@@ -44,7 +44,7 @@ static char posix_mkdir_code[] =
QUOTE( } );
/* Source code for Windows _mkdir. */
-static char win_mkdir_code[] =
+static const char win_mkdir_code[] =
QUOTE( #include <direct.h> )
QUOTE( int main(int argc, char **argv) { )
QUOTE( if (argc != 2) { return 1; } )
@@ -53,7 +53,7 @@ static char win_mkdir_code[] =
QUOTE( } );
/* Source code for rmdir. */
-static char rmdir_code[] =
+static const char rmdir_code[] =
QUOTE( #include <%s> )
QUOTE( int main(int argc, char **argv) { )
QUOTE( if (argc != 2) { return 1; } )
Modified: incubator/lucy/branches/0.1/charmonizer/src/Charmonizer/Core/HeaderChecker.c
URL: http://svn.apache.org/viewvc/incubator/lucy/branches/0.1/charmonizer/src/Charmonizer/Core/HeaderChecker.c?rev=1137407&r1=1137406&r2=1137407&view=diff
==============================================================================
--- incubator/lucy/branches/0.1/charmonizer/src/Charmonizer/Core/HeaderChecker.c (original)
+++ incubator/lucy/branches/0.1/charmonizer/src/Charmonizer/Core/HeaderChecker.c Sun Jun 19 18:40:08 2011
@@ -29,7 +29,7 @@ typedef struct Header {
} Header;
/* "hello_world.c" without the hello or the world. */
-static char test_code[] = "int main() { return 0; }\n";
+static const char test_code[] = "int main() { return 0; }\n";
/* Keep a sorted, dynamically-sized array of names of all headers we've
* checked for so far.
@@ -129,7 +129,7 @@ HeadCheck_check_many_headers(const char
return success;
}
-static char contains_code[] =
+static const char contains_code[] =
QUOTE( #include <stddef.h> )
QUOTE( %s )
QUOTE( int main() { return offsetof(%s, %s); } );
Modified: incubator/lucy/branches/0.1/charmonizer/src/Charmonizer/Core/OperatingSystem.c
URL: http://svn.apache.org/viewvc/incubator/lucy/branches/0.1/charmonizer/src/Charmonizer/Core/OperatingSystem.c?rev=1137407&r1=1137406&r2=1137407&view=diff
==============================================================================
--- incubator/lucy/branches/0.1/charmonizer/src/Charmonizer/Core/OperatingSystem.c (original)
+++ incubator/lucy/branches/0.1/charmonizer/src/Charmonizer/Core/OperatingSystem.c Sun Jun 19 18:40:08 2011
@@ -28,13 +28,13 @@
static char dev_null[20] = "";
#ifdef _WIN32
-static char *exe_ext = ".exe";
-static char *obj_ext = ".obj";
-static char *local_command_start = ".\\";
+static const char *exe_ext = ".exe";
+static const char *obj_ext = ".obj";
+static const char *local_command_start = ".\\";
#else
-static char *exe_ext = "";
-static char *obj_ext = "";
-static char *local_command_start = "./";
+static const char *exe_ext = "";
+static const char *obj_ext = "";
+static const char *local_command_start = "./";
#endif
static void
@@ -68,7 +68,7 @@ S_probe_dev_null(void) {
strcpy(dev_null, "nul");
#else
{
- char *const options[] = {
+ const char *const options[] = {
"/dev/null",
"/dev/nul",
NULL
@@ -109,7 +109,7 @@ OS_dev_null(void) {
return dev_null;
}
-static char charm_run_code[] =
+static const char charm_run_code[] =
QUOTE( #include <stdio.h> )
QUOTE( #include <stdlib.h> )
QUOTE( #include <string.h> )
Modified: incubator/lucy/branches/0.1/charmonizer/src/Charmonizer/Core/Stat.c
URL: http://svn.apache.org/viewvc/incubator/lucy/branches/0.1/charmonizer/src/Charmonizer/Core/Stat.c?rev=1137407&r1=1137406&r2=1137407&view=diff
==============================================================================
--- incubator/lucy/branches/0.1/charmonizer/src/Charmonizer/Core/Stat.c (original)
+++ incubator/lucy/branches/0.1/charmonizer/src/Charmonizer/Core/Stat.c Sun Jun 19 18:40:08 2011
@@ -67,7 +67,7 @@ Stat_stat(const char *filepath, Stat *ta
}
/* Source code for the _charm_stat utility. */
-static char charm_stat_code[] =
+static const char charm_stat_code[] =
QUOTE( #include <stdio.h> )
QUOTE( #include <sys/stat.h> )
QUOTE( int main(int argc, char **argv) { )
Modified: incubator/lucy/branches/0.1/charmonizer/src/Charmonizer/Probe.c
URL: http://svn.apache.org/viewvc/incubator/lucy/branches/0.1/charmonizer/src/Charmonizer/Probe.c?rev=1137407&r1=1137406&r2=1137407&view=diff
==============================================================================
--- incubator/lucy/branches/0.1/charmonizer/src/Charmonizer/Probe.c (original)
+++ incubator/lucy/branches/0.1/charmonizer/src/Charmonizer/Probe.c Sun Jun 19 18:40:08 2011
@@ -69,7 +69,7 @@ Probe_get_charmony_fh(void) {
return ConfWriter_get_charmony_fh();
}
-static char charm_h_code[] =
+static const char charm_h_code[] =
QUOTE( #ifndef CHARM_H )
QUOTE( #define CHARM_H 1 )
QUOTE( #include <stdio.h> )
Modified: incubator/lucy/branches/0.1/charmonizer/src/Charmonizer/Probe/AtomicOps.c
URL: http://svn.apache.org/viewvc/incubator/lucy/branches/0.1/charmonizer/src/Charmonizer/Probe/AtomicOps.c?rev=1137407&r1=1137406&r2=1137407&view=diff
==============================================================================
--- incubator/lucy/branches/0.1/charmonizer/src/Charmonizer/Probe/AtomicOps.c (original)
+++ incubator/lucy/branches/0.1/charmonizer/src/Charmonizer/Probe/AtomicOps.c Sun Jun 19 18:40:08 2011
@@ -25,7 +25,7 @@
#include <stdio.h>
#include <stdlib.h>
-static char osatomic_casptr_code[] =
+static const char osatomic_casptr_code[] =
QUOTE( #include <libkern/OSAtomic.h> )
QUOTE( #include <libkern/OSAtomic.h> )
QUOTE( int main() { )
Modified: incubator/lucy/branches/0.1/charmonizer/src/Charmonizer/Probe/FuncMacro.c
URL: http://svn.apache.org/viewvc/incubator/lucy/branches/0.1/charmonizer/src/Charmonizer/Probe/FuncMacro.c?rev=1137407&r1=1137406&r2=1137407&view=diff
==============================================================================
--- incubator/lucy/branches/0.1/charmonizer/src/Charmonizer/Probe/FuncMacro.c (original)
+++ incubator/lucy/branches/0.1/charmonizer/src/Charmonizer/Probe/FuncMacro.c Sun Jun 19 18:40:08 2011
@@ -25,7 +25,7 @@
#include <stdlib.h>
/* Code for verifying ISO func macro. */
-static char iso_func_code[] =
+static const char iso_func_code[] =
QUOTE( #include "_charm.h" )
QUOTE( int main() { )
QUOTE( Charm_Setup; )
@@ -34,7 +34,7 @@ static char iso_func_code[] =
QUOTE( } );
/* Code for verifying GNU func macro. */
-static char gnuc_func_code[] =
+static const char gnuc_func_code[] =
QUOTE( #include "_charm.h" )
QUOTE( int main() { )
QUOTE( Charm_Setup; )
@@ -43,7 +43,7 @@ static char gnuc_func_code[] =
QUOTE( } );
/* Code for verifying inline keyword. */
-static char inline_code[] =
+static const char inline_code[] =
QUOTE( #include "_charm.h" )
QUOTE( static %s int foo() { return 1; } )
QUOTE( int main() { )
Modified: incubator/lucy/branches/0.1/charmonizer/src/Charmonizer/Probe/Headers.c
URL: http://svn.apache.org/viewvc/incubator/lucy/branches/0.1/charmonizer/src/Charmonizer/Probe/Headers.c?rev=1137407&r1=1137406&r2=1137407&view=diff
==============================================================================
--- incubator/lucy/branches/0.1/charmonizer/src/Charmonizer/Probe/Headers.c (original)
+++ incubator/lucy/branches/0.1/charmonizer/src/Charmonizer/Probe/Headers.c Sun Jun 19 18:40:08 2011
@@ -46,7 +46,7 @@ static void
S_encode_affirmation(const char *header_name);
#define NUM_C89_HEADERS 15
-char *c89_headers[] = {
+const char *c89_headers[] = {
"assert.h",
"ctype.h",
"errno.h",
@@ -66,7 +66,7 @@ char *c89_headers[] = {
};
#define NUM_POSIX_HEADERS 14
-char *posix_headers[] = {
+const char *posix_headers[] = {
"cpio.h",
"dirent.h",
"fcntl.h",
@@ -85,7 +85,7 @@ char *posix_headers[] = {
};
#define NUM_WIN_HEADERS 3
-char *win_headers[] = {
+const char *win_headers[] = {
"io.h",
"windows.h",
"process.h",
Modified: incubator/lucy/branches/0.1/charmonizer/src/Charmonizer/Probe/Integers.c
URL: http://svn.apache.org/viewvc/incubator/lucy/branches/0.1/charmonizer/src/Charmonizer/Probe/Integers.c?rev=1137407&r1=1137406&r2=1137407&view=diff
==============================================================================
--- incubator/lucy/branches/0.1/charmonizer/src/Charmonizer/Probe/Integers.c (original)
+++ incubator/lucy/branches/0.1/charmonizer/src/Charmonizer/Probe/Integers.c Sun Jun 19 18:40:08 2011
@@ -30,7 +30,7 @@
static chaz_bool_t
S_machine_is_big_endian(void);
-static char sizes_code[] =
+static const char sizes_code[] =
QUOTE( #include "_charm.h" )
QUOTE( int main () { )
QUOTE( Charm_Setup; )
@@ -42,7 +42,7 @@ static char sizes_code[] =
QUOTE( return 0; )
QUOTE( } );
-static char type64_code[] =
+static const char type64_code[] =
QUOTE( #include "_charm.h" )
QUOTE( int main() )
QUOTE( { )
@@ -51,7 +51,7 @@ static char type64_code[] =
QUOTE( return 0; )
QUOTE( } );
-static char literal64_code[] =
+static const char literal64_code[] =
QUOTE( #include "_charm.h" )
QUOTE( #define big 9000000000000000000%s )
QUOTE( int main() )
@@ -323,7 +323,7 @@ Integers_run(void) {
/* Create the I64P and U64P printf macros. */
if (has_64) {
int i;
- char *options[] = {
+ const char *options[] = {
"ll",
"l",
"L",
@@ -333,7 +333,7 @@ Integers_run(void) {
};
/* Buffer to hold the code, and its start and end. */
- static char format_64_code[] =
+ static const char format_64_code[] =
QUOTE( #include "_charm.h" )
QUOTE( int main() { )
QUOTE( Charm_Setup; )
Modified: incubator/lucy/branches/0.1/charmonizer/src/Charmonizer/Probe/LargeFiles.c
URL: http://svn.apache.org/viewvc/incubator/lucy/branches/0.1/charmonizer/src/Charmonizer/Probe/LargeFiles.c?rev=1137407&r1=1137406&r2=1137407&view=diff
==============================================================================
--- incubator/lucy/branches/0.1/charmonizer/src/Charmonizer/Probe/LargeFiles.c (original)
+++ incubator/lucy/branches/0.1/charmonizer/src/Charmonizer/Probe/LargeFiles.c Sun Jun 19 18:40:08 2011
@@ -192,7 +192,7 @@ LargeFiles_run(void) {
}
/* Code for checking ftello64 and friends. */
-static char off64_code[] =
+static const char off64_code[] =
QUOTE( %s )
QUOTE( #include "_charm.h" )
QUOTE( int main() { )
@@ -244,7 +244,7 @@ S_probe_off64(off64_combo *combo) {
}
/* Code for checking 64-bit lseek. */
-static char lseek_code[] =
+static const char lseek_code[] =
QUOTE( %s )
QUOTE( #include "_charm.h" )
QUOTE( int main() { )
@@ -287,7 +287,7 @@ S_probe_lseek(unbuff_combo *combo) {
/* Code for checking 64-bit pread. The pread call will fail, but that's fine
* as long as it compiles. */
-static char pread64_code[] =
+static const char pread64_code[] =
QUOTE( %s )
QUOTE( #include "_charm.h" )
QUOTE( int main() { )
@@ -381,7 +381,7 @@ S_test_sparse_file(long offset, Stat *st
}
/* Open a file, seek to a loc, print a char, and communicate success. */
-static char create_bigfile_code[] =
+static const char create_bigfile_code[] =
QUOTE( #include "_charm.h" )
QUOTE( int main() { )
QUOTE( FILE *fh = fopen("_charm_large_file_test", "w+"); )
Modified: incubator/lucy/branches/0.1/charmonizer/src/Charmonizer/Probe/Memory.c
URL: http://svn.apache.org/viewvc/incubator/lucy/branches/0.1/charmonizer/src/Charmonizer/Probe/Memory.c?rev=1137407&r1=1137406&r2=1137407&view=diff
==============================================================================
--- incubator/lucy/branches/0.1/charmonizer/src/Charmonizer/Probe/Memory.c (original)
+++ incubator/lucy/branches/0.1/charmonizer/src/Charmonizer/Probe/Memory.c Sun Jun 19 18:40:08 2011
@@ -25,7 +25,7 @@
#include <stdio.h>
#include <stdlib.h>
-static char alloca_code[] =
+static const char alloca_code[] =
"#include <%s>\n"
QUOTE( int main() { )
QUOTE( void *foo = %s(1); )
@@ -49,7 +49,7 @@ Memory_run(void) {
/* OpenBSD needs sys/types.h for sys/mman.h to work and mmap() to be
* available. Everybody else that has sys/mman.h should have
* sys/types.h as well. */
- char *mman_headers[] = {
+ const char *mman_headers[] = {
"sys/types.h",
"sys/mman.h",
NULL
Modified: incubator/lucy/branches/0.1/charmonizer/src/Charmonizer/Probe/VariadicMacros.c
URL: http://svn.apache.org/viewvc/incubator/lucy/branches/0.1/charmonizer/src/Charmonizer/Probe/VariadicMacros.c?rev=1137407&r1=1137406&r2=1137407&view=diff
==============================================================================
--- incubator/lucy/branches/0.1/charmonizer/src/Charmonizer/Probe/VariadicMacros.c (original)
+++ incubator/lucy/branches/0.1/charmonizer/src/Charmonizer/Probe/VariadicMacros.c Sun Jun 19 18:40:08 2011
@@ -25,7 +25,7 @@
/* Code for verifying ISO-style variadic macros. */
-static char iso_code[] =
+static const char iso_code[] =
QUOTE( #include "_charm.h" )
QUOTE( #define ISO_TEST(fmt, ...) \\ )
" printf(fmt, __VA_ARGS__) \n"
@@ -36,7 +36,7 @@ static char iso_code[] =
QUOTE( } );
/* Code for verifying GNU-style variadic macros. */
-static char gnuc_code[] =
+static const char gnuc_code[] =
QUOTE( #include "_charm.h" )
QUOTE( #define GNU_TEST(fmt, args...) printf(fmt, ##args) )
QUOTE( int main() { )
Modified: incubator/lucy/branches/0.1/charmonizer/src/Charmonizer/Test/TestFuncMacro.c
URL: http://svn.apache.org/viewvc/incubator/lucy/branches/0.1/charmonizer/src/Charmonizer/Test/TestFuncMacro.c?rev=1137407&r1=1137406&r2=1137407&view=diff
==============================================================================
--- incubator/lucy/branches/0.1/charmonizer/src/Charmonizer/Test/TestFuncMacro.c (original)
+++ incubator/lucy/branches/0.1/charmonizer/src/Charmonizer/Test/TestFuncMacro.c Sun Jun 19 18:40:08 2011
@@ -27,7 +27,7 @@ TestFuncMacro_prepare() {
}
#ifdef INLINE
-static INLINE char* S_inline_function() {
+static INLINE const char* S_inline_function() {
return "inline works";
}
#endif