[yast-commit] r49111 - in /trunk/yast2: library/network/src/SuSEFirewall.ycp package/yast2.changes - YaST Commits

15 Jul 2008

Author: locilka
Date: Tue Jul 15 17:33:13 2008
New Revision: 49111

URL: http://svn.opensuse.org/viewcvs/yast?rev=49111&view=rev
Log:
- Fixed ycpdoc warnings for SuSEFirewall (added/fixed docu.)


Modified:
    trunk/yast2/library/network/src/SuSEFirewall.ycp
    trunk/yast2/package/yast2.changes

Modified: trunk/yast2/library/network/src/SuSEFirewall.ycp
URL: http://svn.opensuse.org/viewcvs/yast/trunk/yast2/library/network/src/SuSEFirewall.ycp?rev=49111&r1=49110&r2=49111&view=diff
==============================================================================

--- trunk/yast2/library/network/src/SuSEFirewall.ycp (original)
+++ trunk/yast2/library/network/src/SuSEFirewall.ycp Tue Jul 15 17:33:13 2008
@@ -165,8 +165,10 @@
 	modified = true;
     }
 
-    // Please, do not us this function
-    // Only for firewall installation proposal
+    /**
+     * Do not us this function.
+     * Only for firewall installation proposal
+     */
     global define void ResetModified () {
 	y2milestone("Reseting firewall-modified to 'false'");
 	modified = false;
@@ -255,6 +257,8 @@
     /**
      * Local function returns if other functions should produce verbose output.
      * like popups, reporting errors, etc.
+     *
+     * @return	boolean is_verbose
      */
     boolean IsVerbose () {
 	// verbose level must be above zero to be verbose
@@ -265,6 +269,7 @@
      * Local function for returning default values (if defined) for sysconfig variables.
      *
      * @param	string sysconfig variable
+     * @return	string default value
      */
     string GetDefaultValue (string variable) {
 	return (string) DEFAULT_SETTINGS[variable]:"";
@@ -313,6 +318,7 @@
      * into the disk, use `nil` as the last list item.
      *
      * @param	list <string> of sysconfig variables
+     * @return	boolean if successful
      */
     boolean WriteSysconfigSuSEFirewall (list<string> variables) {
 	boolean write_status = true;
@@ -452,7 +458,7 @@
      * it doesn't return ports that are listed in some service (defined by package)
      * which is enabled.
      *
-     * @return	map  strings are allowed ports or port ranges
+     * @return	map  strings are allowed ports or port ranges
      *
      * @struct $[
      *   "ZONE1" : [ "port1", "port2" ],
@@ -1499,6 +1505,7 @@
      * @struct [ $[ "id":"modem0", "name":"Askey 815C", "type":"dialup", "zone":"EXT" ], ... ]
      *
      * @return	list  >
+     * @return list  > of all interfaces
      */
     global define list  > GetAllKnownInterfaces () {
 	list  > known_interfaces = [];
@@ -1975,6 +1982,13 @@
     /**
      * Returns whether a service is mentioned in FW_CONFIGURATIONS_[EXT|INT|DMZ].
      * These services are defined by random packages.
+     *
+     * @return boolean if service is supported in zone
+     * @param string service, e.g., "service:sshd"
+     * @param string zone, e.g., "EXT"
+     *
+     * @example
+     *	IsServiceDefinedByPackageSupportedInZone ("service:sshd", "EXT") -> true
      */
     boolean IsServiceDefinedByPackageSupportedInZone (string service, string zone) {
 	if (! IsKnownZone(zone)) {
@@ -2069,7 +2083,8 @@
      * Function returns map of supported services all network interfaces.
      *
      * @param	list <string> of services
-     * @return	map  >
+     * @return	map  >
+     *
      * @struct	Returns $[service : $[ interface : supported_status ]]
      *
      * @example
@@ -2115,7 +2130,8 @@
      * Function returns map of supported services in all firewall zones.
      *
      * @param	list <string> of services
-     * @return	map  >
+     * @return	map  >
+     *
      * @struct	Returns $[service : $[ zone_name : supported_status]]
      *
      * @example
@@ -2279,6 +2295,8 @@
     /**
      * Function for reading SuSEFirewall configuration.
      * Fills internal variables only.
+     *
+     * @return boolean if successful
      */
     global define boolean Read () {
 	// Don't fill up the logs with tones of Check-logs
@@ -2551,6 +2569,8 @@
     /**
      * Helper function for the backward compatibility.
      * See WriteConfiguration(). Remove from code ASAP.
+     *
+     * @return boolean if succesful
      */
     global define boolean WriteOnly() {
 	return WriteConfiguration();
@@ -2755,7 +2775,8 @@
     /**
      * Function returns map of `interfaces in zones`.
      *
-     * @return	map 
+     * @return	map  interface in zones
+     *
      * @struct	map $[zone : [list of interfaces]]
      *
      * @example
@@ -2860,7 +2881,8 @@
      * Function returns list of rules of forwarding ports
      * to masqueraded IPs.
      *
-     * @return  list  >
+     * @return  list  > list of rules
+     *
      * @struct	list [$[ key: value ]]
      *
      * @example
@@ -3050,6 +3072,7 @@
      * Function returns yes/no - ingoring broadcast for zone
      *
      * @param	string zone
+     * @return	string "yes" or "no"
      *
      * @example
      *	// Does not logg ignored broadcast packets
@@ -3112,7 +3135,7 @@
 
     // Firewall Expert Rulezz
 
-    /*
+    /**
      * Returns list of rules describing protocols and ports that are allowed
      * to be accessed from listed hosts. All is returned as a single string.
      * Zone needs to be defined.
@@ -3132,7 +3155,7 @@
 	return SETTINGS["FW_SERVICES_ACCEPT_" + zone]:"";
     }
 
-    /*
+    /**
      * Sets expert allow rules for zone.
      *
      * @param string zone

Modified: trunk/yast2/package/yast2.changes
URL: http://svn.opensuse.org/viewcvs/yast/trunk/yast2/package/yast2.changes?rev=49111&r1=49110&r2=49111&view=diff
==============================================================================
--- trunk/yast2/package/yast2.changes (original)
+++ trunk/yast2/package/yast2.changes Tue Jul 15 17:33:13 2008
@@ -3,6 +3,7 @@
 
 - Fixed Linuxrc::SaveInstallInf function to really copy the
   /etc/install.inf at the end of the installation.
+- Fixed ycpdoc warnings for SuSEFirewall (added/fixed docu.)
 
 -------------------------------------------------------------------
 Fri Jul 11 11:11:11 CEST 2008 - locilka@suse.cz

-- 
To unsubscribe, e-mail: yast-commit+unsubscribe@opensuse.org
For additional commands, e-mail: yast-commit+help@opensuse.org

    

[yast-commit] r49111 - in /trunk/yast2: library/network/src/SuSEFirewall.ycp package/yast2.changes

locilka＠svn.opensuse.org

tags

participants (1)