Thanks for the fix!
-K
"William A. Rowe, Jr." <wrowe@rowe-clan.net> writes:
> Thanks for catching this Karl... there were APR_IMPORT_DATA as well
> as APR_EXPORT_DATA symbols - I folded them without realizing that they
> included the 'export' statement. It's gotta stay consistent, so either
> it becomes 'export __declspec(dllimport)' in APR and AP, or it stays
> the way it is. Frankly I think including the export tag in the wrapper
> is a better choice when importing the header, but that could blow unixes
> and was certainly not the path of least resistance.
>
> > -----Original Message-----
> > From: wrowe@locus.apache.org [mailto:wrowe@locus.apache.org]
> > Sent: Monday, October 16, 2000 10:55 PM
> > To: httpd-docs-2.0-cvs@apache.org
> > Subject: cvs commit: apache-2.0/src/lib/apr/include apr_time.h
> >
> >
> > wrowe 00/10/16 20:54:44
> >
> > Modified: src/lib/apr/include apr_time.h
> > Log:
> > Ugh... had different meaning between apr and ap. Not
> > good, all fixed.
> >
> > Submitted by: Karl Fogel <kfogel@collab.net>
> > Reviewed by: Greg Stein
> >
> > Revision Changes Path
> > 1.28 +2 -2 apache-2.0/src/lib/apr/include/apr_time.h
> >
> > Index: apr_time.h
> > ===================================================================
> > RCS file: /home/cvs/apache-2.0/src/lib/apr/include/apr_time.h,v
> > retrieving revision 1.27
> > retrieving revision 1.28
> > diff -u -r1.27 -r1.28
> > --- apr_time.h 2000/10/16 06:18:19 1.27
> > +++ apr_time.h 2000/10/17 03:54:43 1.28
> > @@ -66,8 +66,8 @@
> > * @package APR Time library
> > */
> >
> > -APR_DECLARE_DATA const char apr_month_snames[12][4];
> > -APR_DECLARE_DATA const char apr_day_snames[7][4];
> > +extern APR_DECLARE_DATA const char apr_month_snames[12][4];
> > +extern APR_DECLARE_DATA const char apr_day_snames[7][4];
> >
> > /* number of microseconds since 00:00:00 january 1, 1970 UTC */
> > typedef apr_int64_t apr_time_t;
> >
> >
> >
> >
Received on Sat Oct 21 14:36:11 2006