+1 for me

Although I do remember the reasoning for it, it doesn't mean I like the use of the snmpv3 naming scheme either. So if anyone has any better suggestions for these, I'd still like to hear them.

On Wednesday, December 4, 2002, at 08:49 AM, Derick Rethans wrote:

Hello,

while browsing the CVS I found that the following functions were added
to the CVS recently:

+ PHP_FE(snmpv3get, NULL)
+ PHP_FE(snmpv3walk, NULL)
+ PHP_FE(snmpv3realwalk, NULL)
+ PHP_FALIAS(snmpv3walkoid, snmpv3realwalk, NULL)
+ PHP_FE(snmpv3set, NULL)

But those functionnames don't adhere to our nameing guidelines. As those
are new functions I propose to change them to the following, to be more
consistent with all other functions:

snmpv3get -> snmp3_get
snmpv3walk -> snmp3_walk
snmpv3realwalk -> snmp3_real_walk (or snmp3_walk_oid)
snmpv3set -> snmp3_get

also, there is no need to introduce an alias for a newly created
function so I guess we just should drop it.

I'd like to make those proposed changes ASAP as they are also added in
the PHP_4_3 branch which gets closer to release everyday.

regards,
Derick

--
----------------------------------------------------------------------- --
Derick Rethans http://derickrethans.nl/
JDI Media Solutions http://www.jdimedia.nl/
PHP Magazine - PHP Magazine for Professionals http://php-mag.net/
----------------------------------------------------------------------- --


--
PHP Development Mailing List <http://www.php.net/>
To unsubscribe, visit: http://www.php.net/unsub.php

>---------------------------------------------------------------<
Dan Kalowsky                    "Cause fear is strong and love's
http://www.deadmime.org/~dank    for everyone, who isn't me."
[EMAIL PROTECTED]          - "Burden In My Hand",
[EMAIL PROTECTED]                          Soundgarden


--
PHP Development Mailing List <http://www.php.net/>
To unsubscribe, visit: http://www.php.net/unsub.php

Reply via email to