Mailing List Archive: [patch] modperl 2 docs (APR::OS)

[patch] modperl 2 docs (APR::OS)

Jul 1, 2005, 12:07 PM

Post #1 of 4 (3495 views)

Hello,

On two different pages in the modperl 2 documentation,
the method

APR::OS::current_thread_id()

is listed as

APR::OS::thread_current()

which apparently doesn't exist. Attached is a patch to fix these
cases.

Thanks for a great resource, the modperl docs are fantastic.

Hope this helps,
-E

Re: [patch] modperl 2 docs (APR::OS) [ In reply to ]

e at zacks

Jul 1, 2005, 6:04 PM

Post #2 of 4 (3315 views)

Permalink

Patch is inline this time. Sorry about that.

Thanks,
-E

####

Index: src/docs/2.0/user/handlers/http.pod
===================================================================
--- src/docs/2.0/user/handlers/http.pod (revision 208781)
+++ src/docs/2.0/user/handlers/http.pod (working copy)
@@ -1562,7 +1562,7 @@
require Apache2::MPM;
require APR::OS;
return Apache2::MPM->is_threaded
- ? "$$." . ${ APR::OS::thread_current() }
+ ? "$$." . ${ APR::OS::current_thread_id() }
: $$;
}

Index: src/docs/2.0/user/coding/coding.pod
===================================================================
--- src/docs/2.0/user/coding/coding.pod (revision 208781)
+++ src/docs/2.0/user/coding/coding.pod (working copy)
@@ -178,7 +178,7 @@
use Apache2::MPM ();
if (Apache2::MPM->is_threaded) {
require APR::OS;
- my $tid = APR::OS::thread_current();
+ my $tid = APR::OS::current_thread_id();
print "current thread id: $tid (pid: $$)";
}
else {

---------------------------------------------------------------------
To unsubscribe, e-mail: docs-dev-unsubscribe@perl.apache.org
For additional commands, e-mail: docs-dev-help@perl.apache.org

Re: [patch] modperl 2 docs (APR::OS) [ In reply to ]

stas at stason

Jul 2, 2005, 12:00 PM

Post #3 of 4 (3296 views)

Permalink

Evan A. Zacks wrote:
> Patch is inline this time. Sorry about that.

Thanks Evan, committed.

> - ? "$$." . ${ APR::OS::thread_current() }
> + ? "$$." . ${ APR::OS::current_thread_id() }

--
__________________________________________________________________
Stas Bekman JAm_pH ------> Just Another mod_perl Hacker
http://stason.org/ mod_perl Guide ---> http://perl.apache.org
mailto:stas@stason.org http://use.perl.org http://apacheweek.com
http://modperlbook.org http://apache.org http://ticketmaster.com

---------------------------------------------------------------------
To unsubscribe, e-mail: docs-dev-unsubscribe@perl.apache.org
For additional commands, e-mail: docs-dev-help@perl.apache.org

Re: [patch] modperl 2 docs (APR::OS) [ In reply to ]

stas at stason

Aug 30, 2005, 5:56 PM

Post #4 of 4 (3240 views)

Permalink

(that's an old email that I think never left my outbox, sorry about that)

Evan A. Zacks wrote:
> Patch is inline this time. Sorry about that.

Thanks Evan, committed.

> - ? "$$." . ${ APR::OS::thread_current() }
> + ? "$$." . ${ APR::OS::current_thread_id() }

--
__________________________________________________________________
Stas Bekman JAm_pH ------> Just Another mod_perl Hacker
http://stason.org/ mod_perl Guide ---> http://perl.apache.org
mailto:stas@stason.org http://use.perl.org http://apacheweek.com
http://modperlbook.org http://apache.org http://ticketmaster.com

---------------------------------------------------------------------
To unsubscribe, e-mail: docs-dev-unsubscribe@perl.apache.org
For additional commands, e-mail: docs-dev-help@perl.apache.org

Mailing List Archive

Mailing List Archive

Attached Files: