You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@perl.apache.org by Stas Bekman <st...@stason.org> on 2004/09/17 23:39:49 UTC
APR::OS::thread_current
why APR::OS::thread_current returns an object and to get the tid,
you need to dereference it? that doesn't sounds too
intuitive. Should we change it to return the id right away?
http://perl.apache.org/docs/2.0/api/APR/OS.html#C_thread_current_
--
__________________________________________________________________
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: dev-unsubscribe@perl.apache.org
For additional commands, e-mail: dev-help@perl.apache.org
Re: APR::OS::thread_current
Posted by Stas Bekman <st...@stason.org>.
Philippe M. Chiasson wrote:
>>> but I'd rename the method
>>> to something like APR::OS::thread_current_id() or similar.
>>
>>
>>
>> +1, good idea, gozer!
>>
>> May be just current_tid then? Or even just tid() :)
>
>
> APR::OS::thread_id() ?
I thought there was a point in making it clear that it's a current thread.
So may be your original suggestion of current_thread_id is the best.
--
__________________________________________________________________
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: dev-unsubscribe@perl.apache.org
For additional commands, e-mail: dev-help@perl.apache.org
Re: APR::OS::thread_current
Posted by "Philippe M. Chiasson" <go...@ectoplasm.org>.
Stas Bekman wrote:
> Philippe M. Chiasson wrote:
>
>> Stas Bekman wrote:
>>
>>> why APR::OS::thread_current returns an object and to get the tid,
>>> you need to dereference it? that doesn't sounds too
>>> intuitive. Should we change it to return the id right away?
>>> http://perl.apache.org/docs/2.0/api/APR/OS.html#C_thread_current_
>>
>>
>>
>> I don't see anything wrong with doing that,
>
>
> I wonder why Doug designed it like that in first place.
>
>> but I'd rename the method
>> to something like APR::OS::thread_current_id() or similar.
>
>
> +1, good idea, gozer!
>
> May be just current_tid then? Or even just tid() :)
APR::OS::thread_id() ?
Re: APR::OS::thread_current
Posted by Stas Bekman <st...@stason.org>.
Philippe M. Chiasson wrote:
> Stas Bekman wrote:
>
>> why APR::OS::thread_current returns an object and to get the tid,
>> you need to dereference it? that doesn't sounds too
>> intuitive. Should we change it to return the id right away?
>> http://perl.apache.org/docs/2.0/api/APR/OS.html#C_thread_current_
>
>
> I don't see anything wrong with doing that,
I wonder why Doug designed it like that in first place.
> but I'd rename the method
> to something like APR::OS::thread_current_id() or similar.
+1, good idea, gozer!
May be just current_tid then? Or even just tid() :)
--
__________________________________________________________________
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: dev-unsubscribe@perl.apache.org
For additional commands, e-mail: dev-help@perl.apache.org
Re: APR::OS::thread_current
Posted by "Philippe M. Chiasson" <go...@ectoplasm.org>.
Stas Bekman wrote:
> why APR::OS::thread_current returns an object and to get the tid,
> you need to dereference it? that doesn't sounds too
> intuitive. Should we change it to return the id right away?
> http://perl.apache.org/docs/2.0/api/APR/OS.html#C_thread_current_
I don't see anything wrong with doing that, but I'd rename the method
to something like APR::OS::thread_current_id() or similar.