summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorWez Furlong <wez@php.net>2002-05-21 20:51:31 +0000
committerWez Furlong <wez@php.net>2002-05-21 20:51:31 +0000
commitc05834fbce598759b272fa2aefb0d5aebabe795d (patch)
treef8acd7f7f14a7e935981d54d478fdb47321e3d23
parente2b9f0d964205e052769e5e92056318f707f2067 (diff)
downloadphp-git-c05834fbce598759b272fa2aefb0d5aebabe795d.tar.gz
Reformat some comments.
-rw-r--r--ext/com/COM.c50
-rw-r--r--ext/rpc/com/com_wrapper.c50
2 files changed, 44 insertions, 56 deletions
diff --git a/ext/com/COM.c b/ext/com/COM.c
index f75063cfa6..5e862866f7 100644
--- a/ext/com/COM.c
+++ b/ext/com/COM.c
@@ -519,40 +519,34 @@ PHP_FUNCTION(com_load)
server_name = NULL;
} else if (Z_TYPE_P(server_name) == IS_ARRAY) {
pval **tmp;
- /* DAB: 22 Sept 2001 */
- /* Aha - we have a number of possible */
- /* arguments. They are in the hash */
- /* By name: Server, Domain, Username, Password */
- /* Flags. */
- /* This has been crafted to maintian maximum backward */
- /* compatability, If the server name is specified as a */
- /* string, then the function shoule behave as before */
- /* by defaulting username and password and using the */
- /* (I believe) incorrect CLSCTX_SERVER instantiation */
- /* paramter. However if server is specified in this array */
- /* then we use either CLSCTX_REMOTE_SERVER or whatever */
- /* flags are specified in the array */
+ /* DAB: 22 Sept 2001
+ * Aha - we have a number of possible arguments.
+ * They are in the hash By name: Server, Domain, Username, Password
+ * Flags.
+ * This has been crafted to maintian maximum backward compatability.
+ * If the server name is specified as a string, then the function
+ * should behave as before by defaulting username and password and
+ * using the (I believe) incorrect CLSCTX_SERVER instantiation
+ * paramter. However if server is specified in this array then we
+ * use either CLSCTX_REMOTE_SERVER or whatever flags are specified
+ * in the array */
HashTable *ht = Z_ARRVAL(*server_name);
if (FAILURE == zend_hash_find(ht, "Server", 7, (void **) &tmp)) {
server_name = NULL;
} else {
server_name = *tmp;
convert_to_string_ex(&server_name);
- /* CLSCTX_SERVER includes INPROC and LOCAL */
- /* SERVER. This means that any local server */
- /* will be instantiated BEFORE even looking */
- /* on a remote server. Thus if we have a */
- /* server name, probably we want to access */
- /* a remote machine or we would not have */
- /* bothered specifying it. So it would be */
- /* wrong to to connect locally. Futher, */
- /* unless the name passed is a GUID, there has */
- /* to be something to map the Prog.Id to GUID */
- /* and unless that has been modified to remove */
- /* the information about local instantiation */
- /* CLSCTX_SERVER would force a local instantiation */
- /* This setting can be overridden below if the user */
- /* specifies a flags element */
+ /* CLSCTX_SERVER includes INPROC and LOCAL SERVER. This means
+ * that any local server will be instantiated BEFORE even
+ * looking on a remote server. Thus if we have a server name,
+ * probably we want to access a remote machine or we would not
+ * have bothered specifying it. So it would be wrong to to
+ * connect locally. Futher, unless the name passed is a GUID,
+ * there has to be something to map the Prog.Id to GUID and
+ * unless that has been modified to remove the information
+ * about local instantiation CLSCTX_SERVER would force a local
+ * instantiation This setting can be overridden below if the
+ * user specifies a flags element */
flags = CLSCTX_REMOTE_SERVER;
}
if (FAILURE == zend_hash_find(ht, "Username", 9, (void **) &tmp)) {
diff --git a/ext/rpc/com/com_wrapper.c b/ext/rpc/com/com_wrapper.c
index f75063cfa6..5e862866f7 100644
--- a/ext/rpc/com/com_wrapper.c
+++ b/ext/rpc/com/com_wrapper.c
@@ -519,40 +519,34 @@ PHP_FUNCTION(com_load)
server_name = NULL;
} else if (Z_TYPE_P(server_name) == IS_ARRAY) {
pval **tmp;
- /* DAB: 22 Sept 2001 */
- /* Aha - we have a number of possible */
- /* arguments. They are in the hash */
- /* By name: Server, Domain, Username, Password */
- /* Flags. */
- /* This has been crafted to maintian maximum backward */
- /* compatability, If the server name is specified as a */
- /* string, then the function shoule behave as before */
- /* by defaulting username and password and using the */
- /* (I believe) incorrect CLSCTX_SERVER instantiation */
- /* paramter. However if server is specified in this array */
- /* then we use either CLSCTX_REMOTE_SERVER or whatever */
- /* flags are specified in the array */
+ /* DAB: 22 Sept 2001
+ * Aha - we have a number of possible arguments.
+ * They are in the hash By name: Server, Domain, Username, Password
+ * Flags.
+ * This has been crafted to maintian maximum backward compatability.
+ * If the server name is specified as a string, then the function
+ * should behave as before by defaulting username and password and
+ * using the (I believe) incorrect CLSCTX_SERVER instantiation
+ * paramter. However if server is specified in this array then we
+ * use either CLSCTX_REMOTE_SERVER or whatever flags are specified
+ * in the array */
HashTable *ht = Z_ARRVAL(*server_name);
if (FAILURE == zend_hash_find(ht, "Server", 7, (void **) &tmp)) {
server_name = NULL;
} else {
server_name = *tmp;
convert_to_string_ex(&server_name);
- /* CLSCTX_SERVER includes INPROC and LOCAL */
- /* SERVER. This means that any local server */
- /* will be instantiated BEFORE even looking */
- /* on a remote server. Thus if we have a */
- /* server name, probably we want to access */
- /* a remote machine or we would not have */
- /* bothered specifying it. So it would be */
- /* wrong to to connect locally. Futher, */
- /* unless the name passed is a GUID, there has */
- /* to be something to map the Prog.Id to GUID */
- /* and unless that has been modified to remove */
- /* the information about local instantiation */
- /* CLSCTX_SERVER would force a local instantiation */
- /* This setting can be overridden below if the user */
- /* specifies a flags element */
+ /* CLSCTX_SERVER includes INPROC and LOCAL SERVER. This means
+ * that any local server will be instantiated BEFORE even
+ * looking on a remote server. Thus if we have a server name,
+ * probably we want to access a remote machine or we would not
+ * have bothered specifying it. So it would be wrong to to
+ * connect locally. Futher, unless the name passed is a GUID,
+ * there has to be something to map the Prog.Id to GUID and
+ * unless that has been modified to remove the information
+ * about local instantiation CLSCTX_SERVER would force a local
+ * instantiation This setting can be overridden below if the
+ * user specifies a flags element */
flags = CLSCTX_REMOTE_SERVER;
}
if (FAILURE == zend_hash_find(ht, "Username", 9, (void **) &tmp)) {