vfmg (DEVEL): vfmg - use constants instead of numbers

sparky sparky at pld-linux.org
Tue Nov 1 03:55:22 CET 2005


Author: sparky                       Date: Tue Nov  1 02:55:22 2005 GMT
Module: vfmg                          Tag: DEVEL
---- Log message:
- use constants instead of numbers

---- Files affected:
vfmg:
   vfmg (1.73.2.18 -> 1.73.2.19) 

---- Diffs:

================================================================
Index: vfmg/vfmg
diff -u vfmg/vfmg:1.73.2.18 vfmg/vfmg:1.73.2.19
--- vfmg/vfmg:1.73.2.18	Tue Nov  1 03:13:53 2005
+++ vfmg/vfmg	Tue Nov  1 03:55:16 2005
@@ -765,6 +765,11 @@
 }
 
 # WM functions {{{
+use constant FILENAME	=> 0;
+use constant LOCALENAME	=> 1;
+use constant ICON		=> 2;
+use constant EXEC		=> 3;
+
 $level="";
 sub cmpdname {
 	return $desktop{$a}[1] cmp $desktop{$b}[1];
@@ -774,12 +779,12 @@
 	my $apps="";
 	foreach my $entry (sort cmpdname keys %{$menu[$no]}) {
 		my $d = $desktop{$entry};
-		$name = $$d[1];
+		$name = $$d[LOCALENAME];
 		$name =~ s/\"/\\\"/g;
 		my $icon = "";
-		$icon = scale_icon($$d[2]) if $opt{icons};
+		$icon = scale_icon($$d[ICON]) if $opt{icons};
 		if($menu[$no]{$entry} < 0) {
-			$apps .= qq(${level}prog "$name" "$icon" $$d[3]\n);
+			$apps .= qq(${level}prog "$name" "$icon" $$d[EXEC]\n);
 		} else {
 			$icon = "folder" if not length $icon and $opt{icons};
 			print qq(${level}menu "$name" "$icon" {\n);
@@ -798,9 +803,9 @@
 	foreach my $entry (sort cmpdname keys %{$menu[$no]}) {
 		my $d = $desktop{$entry};
 		if($menu[$no]{$entry} < 0) {
-			$apps .= "$level"."[exec] ($$d[1]) {$$d[3]}\n";
+			$apps .= "$level"."[exec] ($$d[LOCALENAME]) {$$d[EXEC]}\n";
 		} else {
-			print "$level"."[submenu] ($$d[1])\n";
+			print "$level"."[submenu] ($$d[LOCALENAME])\n";
 			$level.="	";
 			blackbox($menu[$no]{$entry});
 			$level=substr($level,1);
@@ -815,10 +820,10 @@
 	my $apps="";
 	foreach my $entry (sort cmpdname keys %{$menu[$no]}) {
 		my $d = $desktop{$entry};
-		my $name = $$d[1];
+		my $name = $$d[LOCALENAME];
 		$name =~ s/\"/\\\"/g;
 		if($menu[$no]{$entry} < 0) {
-			$apps .= qq($level"$name" EXEC $$d[3]\n);
+			$apps .= qq($level"$name" EXEC $$d[EXEC]\n);
 		} else {
 			print qq($level"$name" MENU\n);
 			$level.="	";
@@ -835,10 +840,10 @@
 	my $apps="";
 	foreach my $entry (sort cmpdname keys %{$menu[$no]}) {
 		my $d = $desktop{$entry};
-		$name = $$d[1];
+		$name = $$d[LOCALENAME];
 		$name =~ s/\"/\\\"/g;
 		if($menu[$no]{$entry} < 0) {
-			(my $exec = $$d[3]) =~ s/\"/\\\"/g;
+			(my $exec = $$d[EXEC]) =~ s/\"/\\\"/g;
 			$apps .= "$coma\n".
 				qq{$level("$name", EXEC, "$exec")};
 		} else {
@@ -867,12 +872,13 @@
 	my $apps="";
 	foreach my $entry (sort cmpdname keys %{$menu[$no]}) {
 		my $d = $desktop{$entry};
-		$name = ($$d[1] =~ /["&<>]/) ? xmlname($$d[1]) : $$d[1];
+		$name = ($$d[LOCALENAME] =~ /["&<>]/) ?
+			xmlname($$d[LOCALENAME]) : $$d[LOCALENAME];
 		my $icon = "";
-		$icon = scale_icon($$d[2]) if $opt{icons};
+		$icon = scale_icon($$d[ICON]) if $opt{icons};
 		if($menu[$no]{$entry} < 0) {
 			$apps .= qq($level<app name="$name" cmd=");
-			$apps .= ($$d[3] =~ /["&<>]/) ? xmlname($$d[3]) : $$d[3];
+			$apps .= ($$d[EXEC] =~ /["&<>]/) ? xmlname($$d[EXEC]) : $$d[EXEC];
 			$apps .= qq(" icon="$icon"/>\n);
 		} else {
 			print qq($level<menu name="$name" icon="$icon" visible="yes">\n);
@@ -890,14 +896,14 @@
 	
 	foreach my $entry(keys %{$menu[$no]}) {
 		my $d = $desktop{$entry};
-		$name = encode($opt{encoding},$$d[1]);
+		$name = encode($opt{encoding},$$d[LOCALENAME]);
 		if($menu[$no]{$entry} < 0) {
 			$name=~s/\"/\\\"/g;
 			my $icon = "";
-			$icon = scale_icon($$d[2]) if $opt{icons};
+			$icon = scale_icon($$d[ICON]) if $opt{icons};
 			my $F_OUT;
-			open $F_OUT, ">> $dir/$$d[0]" or warn "$dir/$$d[0]: $!\n";
-			print $F_OUT qq(Exec "$name" exec $$d[3]\n);
+			open $F_OUT, ">> $dir/$$d[FILENAME]" or warn "$dir/$$d[FILENAME]: $!\n";
+			print $F_OUT qq(Exec "$name" exec $$d[EXEC]\n);
 			print $F_OUT qq(MiniPixmap "$icon"\n) if length $icon;
 			close $F_OUT;
 		} else {
@@ -918,14 +924,14 @@
 	foreach my $entry (sort cmpdname keys %{$menu[$no]}) {
 		my $d = $desktop{$entry};
 		if($menu[$no]{$entry} < 0) {
-			$name = $$d[1];
+			$name = $$d[LOCALENAME];
 			$name =~ s/\"/\\\"/g;
-			$apps .= qq(\tExec "$name"\texec $$d[3]\n);
+			$apps .= qq(\tExec "$name"\texec $$d[EXEC]\n);
 		} else {
-			$name = $$d[0];
+			$name = $$d[FILENAME];
 			$name =~ s/\s+/_/g;
-			$this_menu .= qq(\tPopup "$$d[1]"\t$file.$name\n);
-			fvwm($menu[$no]{$entry},"$file.$name",$$d[1]);
+			$this_menu .= qq(\tPopup "$$d[LOCALENAME]"\t$file.$name\n);
+			fvwm($menu[$no]{$entry},"$file.$name",$$d[LOCALENAME]);
 		}
 	}
 	print $this_menu . $apps . "EndPopup\n\n" if length $file;
@@ -936,14 +942,14 @@
 	my $apps="";
 	foreach my $entry (sort cmpdname keys %{$menu[$no]}) {
 		my $d = $desktop{$entry};
-		( my $name = $$d[1] ) =~ s/\"/\\\"/g;
+		( my $name = $$d[LOCALENAME] ) =~ s/\"/\\\"/g;
 		if($menu[$no]{$entry}<0) {
 			my $icon = "";
-			$icon = scale_icon($$d[2]) if $opt{icons};
-			$apps .= qq(AddToMenu $level	"$name"	Exec $$d[3] &\n);
+			$icon = scale_icon($$d[ICON]) if $opt{icons};
+			$apps .= qq(AddToMenu $level	"$name"	Exec $$d[EXEC] &\n);
 			$apps .= qq(Style "$name"	Icon $icon\n) if length $icon;
 		} else {
-			my $file = $$d[0];
+			my $file = $$d[FILENAME];
 			$file =~ s/\s+/_/g;
 			$level .= ".$file";
 			print qq(AddToMenu $level	"$name"	Popup $level\n);
@@ -966,16 +972,16 @@
 	foreach my $entry (sort cmpdname keys %{$menu[$no]}) {
 		my $d = $desktop{$entry};
 		my $icon = "";
-		$icon = scale_icon($$d[2]) if $opt{icons};
+		$icon = scale_icon($$d[ICON]) if $opt{icons};
 		$icon = "\%$icon\%" if length $icon;
 		
 		if($menu[$no]{$entry} < 0) {
-			( my $name = $$d[1] ) =~ s/\"/\\\"/g;
-			$apps .= qq(+ "$icon$name"\tExec exec $$d[3]\n);
+			( my $name = $$d[LOCALENAME] ) =~ s/\"/\\\"/g;
+			$apps .= qq(+ "$icon$name"\tExec exec $$d[EXEC]\n);
 		} else {
-			( my $name = $$d[0] ) =~ s/\s+/_/g;
-			$this_menu .= qq(+ "$icon$$d[1]"\tPopup\t$file.$name\n);
-			metisse($menu[$no]{$entry},"$file.$name",$$d[1]);
+			( my $name = $$d[FILENAME] ) =~ s/\s+/_/g;
+			$this_menu .= qq(+ "$icon$$d[LOCALENAME]"\tPopup\t$file.$name\n);
+			metisse($menu[$no]{$entry},"$file.$name",$$d[LOCALENAME]);
 		}
 	}
 	print $this_menu . $apps . "\n" if length $file;
@@ -986,9 +992,9 @@
 	my $apps="";
 	foreach my $entry (sort cmpdname keys %{$menu[$no]}) {
 		my $d = $desktop{$entry};
-		( my $name = $$d[1] ) =~ s/\"/\\\"/g;
+		( my $name = $$d[LOCALENAME] ) =~ s/\"/\\\"/g;
 		if($menu[$no]{$entry}<0) {
-			$apps .= qq($level"$name"	exec $$d[3]\n);
+			$apps .= qq($level"$name"	exec $$d[EXEC]\n);
 		} else {
 			print qq($level"$name" MENU\n);
 			$level.="	";
@@ -1005,11 +1011,11 @@
 	my $apps="";
 	foreach my $entry (sort cmpdname keys %{$menu[$no]}) {
 		my $d = $desktop{$entry};
-		( my $name = $$d[1] ) =~ s/\"/\\\"/g;
+		( my $name = $$d[LOCALENAME] ) =~ s/\"/\\\"/g;
 		my $icon = "";
-		$icon = scale_icon($$d[2]) if $opt{icons};
+		$icon = scale_icon($$d[ICON]) if $opt{icons};
 		if($menu[$no]{$entry} < 0) {
-			( my $exec = $$d[3] ) =~ s/\"/\\\"/g;
+			( my $exec = $$d[EXEC] ) =~ s/\"/\\\"/g;
 			$apps .= qq($level"$name" "$icon" "$exec"\n);
 		} else {
 			print qq($level"$name" "$icon"\n),
@@ -1029,9 +1035,9 @@
 	my $apps="";
 	foreach my $entry (sort cmpdname keys %{$menu[$no]}) {
 		my $d = $desktop{$entry};
-		( my $name = $$d[1] ) =~ s/\"/\\\"/g;
+		( my $name = $$d[LOCALENAME] ) =~ s/\"/\\\"/g;
 		if($menu[$no]{$entry} < 0) {
-			( my $exec = $$d[3] ) =~ s/\"/\\\"/g;
+			( my $exec = $$d[EXEC] ) =~ s/\"/\\\"/g;
 			$apps .= qq(${level}cmd "$name" "$exec"\n);
 		} else {
 			print qq(${level}menu "$name"\n);
@@ -1052,15 +1058,15 @@
 	my $apps="";
 	foreach my $entry (sort cmpdname keys %{$menu[$no]}) {
 		my $d = $desktop{$entry};
-		$name=encode($opt{encoding},$$d[1]);
+		$name=encode($opt{encoding},$$d[LOCALENAME]);
 		$name=~s/\"/\'/g;
 		my $icon="";
-		$icon = scale_icon($$d[2]) if $opt{icons};
+		$icon = scale_icon($$d[ICON]) if $opt{icons};
 		if($menu[$no]{$entry}<0) {
-			$apps .= qq("$name" "$icon" exec "$$d[3]"\n);
+			$apps .= qq("$name" "$icon" exec "$$d[EXEC]"\n);
 		} else {
-			print $F_OUT qq("$name" "$icon" menu "$dir/$$d[0].menu"\n);
-			enlightenment($menu[$no]{$entry},$$d[0],$name);
+			print $F_OUT qq("$name" "$icon" menu "$dir/$$d[FILENAME].menu"\n);
+			enlightenment($menu[$no]{$entry},$$d[FILENAME],$name);
 		}
 	}
 	print $F_OUT $apps;
@@ -1071,15 +1077,15 @@
 	my ($no, $dir)=@_;
 	foreach my $entry (keys %{$menu[$no]}) {
 		my $d = $desktop{$entry};
-		$name = encode($opt{encoding},$$d[1]);
+		$name = encode($opt{encoding},$$d[LOCALENAME]);
 		if($menu[$no]{$entry} < 0) {
 			$name =~ s/\"/\\\"/g;
 			my $icon = "";
-			$icon = scale_icon($$d[2]) if $opt{icons};
-			open F_OUT, ">> $dir/$$d[0].lnk" or warn "$dir/$$d[0].lnk: $!\n";
+			$icon = scale_icon($$d[ICON]) if $opt{icons};
+			open F_OUT, ">> $dir/$$d[FILENAME].lnk" or warn "$dir/$$d[FILENAME].lnk: $!\n";
 			print F_OUT "[Shortcut]\n",
 						"Caption=$name\n",
-						"Command=$$d[3]\n";
+						"Command=$$d[EXEC]\n";
 			print F_OUT "Icon=$icon\n" if length $icon;
 			close F_OUT;
 		} else {
@@ -1096,17 +1102,17 @@
 	foreach my $entry (sort cmpdname keys %{$menu[$no]}) {
 		my $d = $desktop{$entry};
 		my $icon = "";
-		$icon = scale_icon($$d[2]) if $opt{icons};
+		$icon = scale_icon($$d[ICON]) if $opt{icons};
 		if($menu[$no]{$entry}<0) {
 			$apps .=
 				"${level}item {\n".
-				"${level}	name = $$d[1]\n".
-				"${level}	action = $$d[3]\n".
+				"${level}	name = $$d[LOCALENAME]\n".
+				"${level}	action = $$d[EXEC]\n".
 				"${level}	image = $icon\n".
 				"${level}}\n";
 		} else {
 			print "${level}menu {\n";
-			print "${level}	name = $$d[1]\n";
+			print "${level}	name = $$d[LOCALENAME]\n";
 			print "${level}	image = $icon\n";
 			$level.="	";
 			fbpanel($menu[$no]{$entry});
@@ -1124,13 +1130,13 @@
 		my $d = $desktop{$entry};
 		if($menu[$no]{$entry}<0) {
 			$apps .=
-				qq($level<item label="$$d[1]">\n).
+				qq($level<item label="$$d[LOCALENAME]">\n).
 				qq($level	<action name="Execute">\n).
-				qq($level	 <execute>$$d[3]</execute>\n).
+				qq($level	 <execute>$$d[EXEC]</execute>\n).
 				qq($level	</action>\n).
 				qq($level</item>\n);
 		} else {
-			print qq($level<menu id="$$d[1]" label="$$d[1]">\n);
+			print qq($level<menu id="$$d[LOCALENAME]" label="$$d[LOCALENAME]">\n);
 			$level.="	";
 			openbox($menu[$no]{$entry});
 			$level=substr($level,1);
================================================================

---- CVS-web:
    http://cvs.pld-linux.org/vfmg/vfmg?r1=1.73.2.18&r2=1.73.2.19&f=u




More information about the pld-cvs-commit mailing list