You are viewing a plain text version of this content. The canonical link for it is here.
Posted to modperl-cvs@perl.apache.org by to...@apache.org on 2010/08/22 19:51:03 UTC
svn commit: r987933 - in /perl/modperl/trunk: t/response/TestAPR/socket.pm
xs/APR/Socket/APR__Socket.h xs/maps/apr_functions.map
Author: torsten
Date: Sun Aug 22 17:51:03 2010
New Revision: 987933
URL: http://svn.apache.org/viewvc?rev=987933&view=rev
Log:
make APR::Socket::fileno unimplemented on Windows
Modified:
perl/modperl/trunk/t/response/TestAPR/socket.pm
perl/modperl/trunk/xs/APR/Socket/APR__Socket.h
perl/modperl/trunk/xs/maps/apr_functions.map
Modified: perl/modperl/trunk/t/response/TestAPR/socket.pm
URL: http://svn.apache.org/viewvc/perl/modperl/trunk/t/response/TestAPR/socket.pm?rev=987933&r1=987932&r2=987933&view=diff
==============================================================================
--- perl/modperl/trunk/t/response/TestAPR/socket.pm (original)
+++ perl/modperl/trunk/t/response/TestAPR/socket.pm Sun Aug 22 17:51:03 2010
@@ -40,14 +40,11 @@ sub handler {
$socket->timeout_set($orig_val);
ok t_cmp($socket->timeout_get(), $orig_val, "timeout_get()");
- my $fd=$socket->fileno;
- t_debug "client socket fd=$fd";
- if ($^O eq 'MSWin32') {
- ok $fd==-1;
- }
- else {
- ok $fd>=0;
- }
+ skip $^O=~/mswin/i ? 'APR::Socket->fileno is not implemented on MSWin' : '',
+ sub {
+ t_debug "client socket fd=".$socket->fileno;
+ $socket->fileno>0
+ };
Apache2::Const::OK;
}
Modified: perl/modperl/trunk/xs/APR/Socket/APR__Socket.h
URL: http://svn.apache.org/viewvc/perl/modperl/trunk/xs/APR/Socket/APR__Socket.h?rev=987933&r1=987932&r2=987933&view=diff
==============================================================================
--- perl/modperl/trunk/xs/APR/Socket/APR__Socket.h (original)
+++ perl/modperl/trunk/xs/APR/Socket/APR__Socket.h Sun Aug 22 17:51:03 2010
@@ -117,13 +117,11 @@ apr_status_t mpxs_APR__Socket_poll(apr_s
return apr_poll(&fd, 1, &nsds, timeout);
}
+#ifndef WIN32
static MP_INLINE int mpxs_APR__Socket_fileno(pTHX_ apr_socket_t *sock)
{
-#ifdef WIN32
- return -1; /* not implemented */
-#else
apr_os_sock_t s;
apr_os_sock_get(&s, sock);
return s;
-#endif
}
+#endif
Modified: perl/modperl/trunk/xs/maps/apr_functions.map
URL: http://svn.apache.org/viewvc/perl/modperl/trunk/xs/maps/apr_functions.map?rev=987933&r1=987932&r2=987933&view=diff
==============================================================================
--- perl/modperl/trunk/xs/maps/apr_functions.map (original)
+++ perl/modperl/trunk/xs/maps/apr_functions.map Sun Aug 22 17:51:03 2010
@@ -55,7 +55,9 @@ MODULE=APR::Socket
!apr_socket_shutdown
-apr_socket_inherit_set
-apr_socket_inherit_unset
+#_if_ $^O !~ /mswin/i
mpxs_APR__Socket_fileno | | apr_socket_t *:socket
+#_end_
MODULE=APR::Socket
apr_socket_close
@@ -627,7 +629,11 @@ MODULE=APR::OS
-apr_os_exp_time_get
-apr_os_file_get
-apr_os_imp_time_get
+#_if_ $^O !~ /mswin/i
~apr_os_sock_get
+#_else_
+-apr_os_sock_get
+#_end_
-apr_os_thread_get
-apr_os_threadkey_get
-apr_os_sock_make