Skip to content

Commit 6083801

Browse files
author
Brian Pane
committed
Moved the definition of apr_table_t's internals from apr_tables.h
to apr_tables.c to strengthen its encapsulation git-svn-id: https://svn.apache.org/repos/asf/apr/apr/trunk@63570 13f79535-47bb-0310-9956-ffa450edef68
1 parent 74963ad commit 6083801

File tree

2 files changed

+15
-15
lines changed

2 files changed

+15
-15
lines changed

include/apr_tables.h

-15
Original file line numberDiff line numberDiff line change
@@ -102,21 +102,6 @@ struct apr_array_header_t {
102102
char *elts;
103103
};
104104

105-
/** The opaque string-content table type */
106-
struct apr_table_t {
107-
/* This has to be first to promote backwards compatibility with
108-
* older modules which cast a apr_table_t * to an apr_array_header_t *...
109-
* they should use the table_elts() function for most of the
110-
* cases they do this for.
111-
*/
112-
/** The underlying array for the table */
113-
apr_array_header_t a;
114-
#ifdef MAKE_TABLE_PROFILE
115-
/** Who created the array. */
116-
void *creator;
117-
#endif
118-
};
119-
120105
/**
121106
* The (opaque) structure for string-content tables.
122107
*/

tables/apr_tables.c

+15
Original file line numberDiff line numberDiff line change
@@ -337,6 +337,21 @@ APR_DECLARE(char *) apr_array_pstrcat(apr_pool_t *p,
337337
checksum &= CASE_MASK; \
338338
}
339339

340+
/** The opaque string-content table type */
341+
struct apr_table_t {
342+
/* This has to be first to promote backwards compatibility with
343+
* older modules which cast a apr_table_t * to an apr_array_header_t *...
344+
* they should use the table_elts() function for most of the
345+
* cases they do this for.
346+
*/
347+
/** The underlying array for the table */
348+
apr_array_header_t a;
349+
#ifdef MAKE_TABLE_PROFILE
350+
/** Who created the array. */
351+
void *creator;
352+
#endif
353+
};
354+
340355
/*
341356
* XXX: if you tweak this you should look at is_empty_table() and table_elts()
342357
* in alloc.h

0 commit comments

Comments
 (0)