From: cigamit <cigamit@7612ce4b-ef26-0410-bec9-ea0150e637f0>
Date: Mon, 1 Nov 2004 02:05:15 +0000 (+0000)
Subject: Removed variables that were initialized, but never actually used
X-Git-Url: https://vcs.fsf.org/?p=squirrelmail.git;a=commitdiff_plain;h=8d8da447778a43b78bc95f9601b385416ad84477

Removed variables that were initialized, but never actually used
Also remove a few more globals that were unused
Remove a few newlines at the end of files
Hopefully this will help shrink the memory footprint of SM just slightly


git-svn-id: https://svn.code.sf.net/p/squirrelmail/code/trunk/squirrelmail@8296 7612ce4b-ef26-0410-bec9-ea0150e637f0
---

diff --git a/class/deliver/Deliver.class.php b/class/deliver/Deliver.class.php
index 6adaf3da..093f5a27 100644
--- a/class/deliver/Deliver.class.php
+++ b/class/deliver/Deliver.class.php
@@ -163,7 +163,6 @@ class Deliver {
             } elseif ($message->att_local_name) {
                 $filename = $message->att_local_name;
                 $file = fopen ($filename, 'rb');
-                $encoded = '';
                 while ($tmp = fread($file, 570)) {
                    $body_part = chunk_split(base64_encode($tmp));
                     $length += $this->clean_crlf($body_part);
@@ -308,7 +307,6 @@ class Deliver {
             $header[] .= 'Content-Description: ' . $mime_header->description . $rn;
         }
         if ($mime_header->encoding) {
-            $encoding = $mime_header->encoding;
             $header[] .= 'Content-Transfer-Encoding: ' . $mime_header->encoding . $rn;
         } else {
             if ($mime_header->type0 == 'text' || $mime_header->type0 == 'message') {
diff --git a/class/deliver/Deliver_SMTP.class.php b/class/deliver/Deliver_SMTP.class.php
index 72b6b6f7..1042d383 100644
--- a/class/deliver/Deliver_SMTP.class.php
+++ b/class/deliver/Deliver_SMTP.class.php
@@ -347,4 +347,4 @@ class Deliver_SMTP extends Deliver {
     }
 }
 
-?>
+?>
\ No newline at end of file
diff --git a/class/helper/VCard.class.php b/class/helper/VCard.class.php
index 022607c1..6dd5dd0e 100644
--- a/class/helper/VCard.class.php
+++ b/class/helper/VCard.class.php
@@ -32,4 +32,4 @@ return $array;
 
 }
 
-?>
+?>
\ No newline at end of file
diff --git a/class/html.class.php b/class/html.class.php
index edc99a49..fba98feb 100644
--- a/class/html.class.php
+++ b/class/html.class.php
@@ -172,4 +172,4 @@ class html {
 }
  
 
-?>
+?>
\ No newline at end of file
diff --git a/class/mime/AddressStructure.class.php b/class/mime/AddressStructure.class.php
index f5fecc99..b3201e23 100644
--- a/class/mime/AddressStructure.class.php
+++ b/class/mime/AddressStructure.class.php
@@ -63,4 +63,4 @@ class AddressStructure {
     }
 }
 
-?>
+?>
\ No newline at end of file
diff --git a/class/mime/ContentType.class.php b/class/mime/ContentType.class.php
index 79ca37fb..c9b278eb 100644
--- a/class/mime/ContentType.class.php
+++ b/class/mime/ContentType.class.php
@@ -33,4 +33,4 @@ class ContentType {
     }
 }
 
-?>
+?>
\ No newline at end of file
diff --git a/class/mime/Disposition.class.php b/class/mime/Disposition.class.php
index 38f8d292..71feef8a 100644
--- a/class/mime/Disposition.class.php
+++ b/class/mime/Disposition.class.php
@@ -31,4 +31,4 @@ class Disposition {
     }
 }
 
-?>
+?>
\ No newline at end of file
diff --git a/class/mime/Message.class.php b/class/mime/Message.class.php
index 58d01458..94e59c3e 100644
--- a/class/mime/Message.class.php
+++ b/class/mime/Message.class.php
@@ -48,7 +48,6 @@ class Message {
     }
 
     function getFilename() {
-        $filename = '';
         $filename = $this->header->getParameter('filename');
         if (!$filename) {
             $filename = $this->header->getParameter('name');
diff --git a/class/mime/Rfc822Header.class.php b/class/mime/Rfc822Header.class.php
index 921b22cc..e8097976 100644
--- a/class/mime/Rfc822Header.class.php
+++ b/class/mime/Rfc822Header.class.php
@@ -227,7 +227,6 @@ class Rfc822Header {
 
     function getAddressTokens($address) {
         $aTokens = array();
-        $aAddress = array();
         $aSpecials = array('(' ,'<' ,',' ,';' ,':');
         $aReplace =  array(' (',' <',' ,',' ;',' :');
         $address = str_replace($aSpecials,$aReplace,$address);
@@ -412,7 +411,7 @@ class Rfc822Header {
 
     function parseAddress($address,$ar=false,$aAddress=array(),$sGroup='',$sHost='',$lookup=false) {
         $aTokens = $this->getAddressTokens($address);
-        $sPersonal = $sEmail = $sComment = $sGroup = '';
+        $sPersonal = $sEmail = $sGroup = '';
         $aStack = $aComment = array();
         foreach ($aTokens as $sToken) {
             $cChar = $sToken{0};
@@ -621,7 +620,7 @@ class Rfc822Header {
         if (is_array($arr)) {
             foreach($arr as $arg) {
                 if ($this->getAddr_s($arg, $separator, $encoded)) {
-                    $s .= $separator . $result;
+                    $s .= $separator;
                 }
             }
             $s = ($s ? substr($s, 2) : $s);
@@ -692,7 +691,6 @@ class Rfc822Header {
             $i=0;
             foreach($address as $argument) {
                 $match = $this->findAddress($argument, true);
-                $last = end($match);
                 if ($match[1]) {
                     return $i;
                 } else {
diff --git a/functions/decode/utf_8.php b/functions/decode/utf_8.php
index 968f8924..c3bd951f 100644
--- a/functions/decode/utf_8.php
+++ b/functions/decode/utf_8.php
@@ -27,7 +27,7 @@
  * @return string Decoded string
  */
 function charset_decode_utf_8 ($string) {
-  global $default_charset,$squirrelmail_language;
+  global $squirrelmail_language;
 
     if ($squirrelmail_language == 'ja_JP')
         return $string;
diff --git a/functions/html.php b/functions/html.php
index 91acef95..92c21b1c 100644
--- a/functions/html.php
+++ b/functions/html.php
@@ -93,7 +93,6 @@ function html_tag( $tag,                // Tag to output
     /* especially usefull when $url = $PHP_SELF */
     function set_url_var($url, $var, $val=0, $link=true) {
         $k = '';
-        $ret = '';
         $pat_a = array (
                        '/.+(\\&'.$var.')=(.*)\\&/AU',   /* in the middle */
                        '/.+\\?('.$var.')=(.*\\&).+/AU', /* at front, more follow */
diff --git a/functions/i18n.php b/functions/i18n.php
index 8e434c24..3528c1cf 100644
--- a/functions/i18n.php
+++ b/functions/i18n.php
@@ -305,6 +305,7 @@ function set_up_language($sm_language, $do_search = false, $default = false) {
         if ($sm_notAlias=='tr_TR') setlocale(LC_CTYPE,'C');
 
         // Set text direction/alignment variables
+        // These don't appear to be used... are they safe to remove?
         if (isset($languages[$sm_notAlias]['DIR']) &&
             $languages[$sm_notAlias]['DIR'] == 'rtl') {
             /**
@@ -542,6 +543,7 @@ function japanese_xtra_wordwrap($ret,$wrap) {
             "\xc3\xa5\xe3\xa5\xe5\xa5\xe7\xa5\xee\xa5\xf5\xa5\xf6\xa1\xa6\xa1\xbc" .
             "\xa1\xb3\xa1\xb4\xa1\xaa\xa1\xf3\xa1\xcb\xa1\xa4\xa1\xa5\xa1\xa7\xa1" .
             "\xa8\xa1\xa9\xa1\xcf\xa1\xd1";
+        // This don't appear to be used... is it safe to remove?
         $no_end = "\x5c\x24\x28\x5b\x7b\xa1\xf2\x5c\xa1\xc6\xa1\xc8\xa1\xd2\xa1" .
             "\xd4\xa1\xd6\xa1\xd8\xa1\xda\xa1\xcc\xa1\xf0\xa1\xca\xa1\xce\xa1\xd0\xa1\xef";
 
diff --git a/functions/imap_asearch.php b/functions/imap_asearch.php
index a1238c37..c60b8d10 100644
--- a/functions/imap_asearch.php
+++ b/functions/imap_asearch.php
@@ -446,7 +446,6 @@ function sqimap_asearch($imapConnection, &$mailbox_array, &$biop_array, &$unop_a
 {
 
     $search_charset = sqimap_asearch_get_charset();
-    $mbox_msgs = array();
     $mbox_search = array();
     $search_string = '';
     $cur_mailbox = $mailbox_array[0];
diff --git a/functions/imap_general.php b/functions/imap_general.php
index 0a70f685..8a117dec 100755
--- a/functions/imap_general.php
+++ b/functions/imap_general.php
@@ -320,7 +320,7 @@ function sqimap_retrieve_imap_response($imap_stream, $tag, $handle_errors,
     $resultlist = array();
     $data = array();
     $read = sqimap_fgets($imap_stream);
-    $i = $k = 0;
+    $i = 0;
     while ($read) {
         $char = $read{0};
         switch ($char)
diff --git a/functions/imap_mailbox.php b/functions/imap_mailbox.php
index ce3503e6..117cbe41 100755
--- a/functions/imap_mailbox.php
+++ b/functions/imap_mailbox.php
@@ -559,14 +559,11 @@ function sqimap_mailbox_option_list($imap_stream, $show_selected = 0, $folder_sk
 
 
 function sqimap_mailbox_list($imap_stream, $force=false) {
-    global $default_folder_prefix;
-
     if (!sqgetGlobalVar('boxesnew',$boxesnew,SQ_SESSION) || $force) {
         global $data_dir, $username, $list_special_folders_first,
                $folder_prefix, $trash_folder, $sent_folder, $draft_folder,
                $move_to_trash, $move_to_sent, $save_as_draft,
                $delimiter, $noselect_fix_enable, $imap_server_type;
-        $inbox_in_list = false;
         $inbox_subscribed = false;
         $listsubscribed = sqimap_capability($imap_stream,'LIST-SUBSCRIBED');
 
@@ -689,7 +686,6 @@ function sqimap_mailbox_list_all($imap_stream) {
     $read_ary = compact_mailboxes_response($read_ary);
 
     $g = 0;
-    $phase = 'inbox';
     $fld_pre_length = strlen($folder_prefix);
     for ($i = 0, $cnt = count($read_ary); $i < $cnt; $i++) {
         /* Store the raw IMAP reply */
@@ -755,9 +751,6 @@ function sqimap_mailbox_tree($imap_stream) {
                $folder_prefix, $delimiter, $trash_folder, $move_to_trash,
                $imap_server_type;
 
-
-        $inbox_in_list = false;
-        $inbox_subscribed = false;
         $noselect = false;
         $noinferiors = false;
 
@@ -862,7 +855,7 @@ function sqimap_fill_mailbox_tree($mbx_ary, $mbxs=false,$imap_stream) {
            $move_to_trash, $move_to_sent, $save_as_draft,
            $delimiter, $imap_server_type;
 
-    $special_folders = array ('INBOX', $sent_folder, $draft_folder, $trash_folder);
+    // $special_folders = array ('INBOX', $sent_folder, $draft_folder, $trash_folder);
 
     /* create virtual root node */
     $mailboxes= new mailboxes();
@@ -874,7 +867,6 @@ function sqimap_fill_mailbox_tree($mbx_ary, $mbxs=false,$imap_stream) {
     if (isset($folder_prefix) && ($folder_prefix != '')) {
         $start = substr_count($folder_prefix,$delimiter);
         if (strrpos($folder_prefix, $delimiter) == (strlen($folder_prefix)-1)) {
-            $trail_del = true;
             $mailboxes->mailboxname_full = substr($folder_prefix,0, (strlen($folder_prefix)-1));
         } else {
             $mailboxes->mailboxname_full = $folder_prefix;
@@ -988,7 +980,7 @@ function sqimap_tree_to_ref_array(&$mbx_tree,&$aMbxs) {
 
 function sqimap_get_status_mbx_tree($imap_stream,&$mbx_tree) {
     global $unseen_notify, $unseen_type, $trash_folder,$move_to_trash;
-    $aMbxs = $aQuery = $aTag = array();
+    $aMbxs = $aQuery = array();
     sqimap_tree_to_ref_array($mbx_tree,$aMbxs);
     // remove the root node
     array_shift($aMbxs);
diff --git a/functions/imap_messages.php b/functions/imap_messages.php
index 6a6c0767..de966785 100755
--- a/functions/imap_messages.php
+++ b/functions/imap_messages.php
@@ -133,10 +133,6 @@ function sqimap_message_list_squisher($messages_array) {
 function sqimap_get_sort_order($imap_stream, $sSortField, $reverse, $search='ALL') {
     global  $default_charset;
 
-    $id = array();
-    $sort_test = array();
-    $sort_query = '';
-
     if ($sSortField) {
         if ($reverse) {
             $sSortField = 'REVERSE '.$sSortField;
@@ -429,7 +425,6 @@ function get_thread_sort($imap_stream, $search='ALL') {
         $thread_temp = preg_split("//", $thread_list, -1, PREG_SPLIT_NO_EMPTY);
     }
 
-    $char_count = count($thread_temp);
     $counter = 0;
     $thread_new = array();
     $k = 0;
@@ -569,7 +564,6 @@ function sqimap_get_small_header_list($imap_stream, $msg_list,
     $aFetchItems = array('FLAGS', 'RFC822.SIZE', 'INTERNALDATE')) {
 
     $aMessageList = array();
-    $read_list = array();
 
     $bUidFetch = ! in_array('UID', $aFetchItems, true);
 
@@ -588,7 +582,6 @@ function sqimap_get_small_header_list($imap_stream, $msg_list,
         }
     } else {
         $msgs_str = '1:*';
-        $aId = array();
     }
 
     /*
diff --git a/functions/imap_search.php b/functions/imap_search.php
index 90f288a9..ea9cfda9 100644
--- a/functions/imap_search.php
+++ b/functions/imap_search.php
@@ -29,8 +29,6 @@ function sqimap_search($imapConnection, $search_where, $search_what, $mailbox,
 
     $pos = $search_position;
 
-    $urlMailbox = urlencode($mailbox);
-
     /* construct the search query, taking multiple search terms into account */
     $multi_search = array();
     $search_what  = trim($search_what);
@@ -116,7 +114,6 @@ function sqimap_search($imapConnection, $search_where, $search_what, $mailbox,
     for ($q = 0; $q < $cnt; $q++) {
         $id[$q] = trim($messagelist[$q]);
     }
-    $issent = ($mailbox == $sent_folder);
 
     $msgs = fillMessageArray($imapConnection,$id,$cnt);
 
diff --git a/functions/mailbox_display.php b/functions/mailbox_display.php
index 3e497743..32e5d40c 100644
--- a/functions/mailbox_display.php
+++ b/functions/mailbox_display.php
@@ -124,6 +124,8 @@ function printMessageInfo($aMsg) {
     $aFlags   = (isset($msg['FLAGS'])) ? $msg['FLAGS'] : array();
     $iPrio    = (isset($msg['PRIORITY'])) ? $msg['PRIORITY'] : 3;
     $iSize    = (isset($msg['SIZE'])) ? $msg['SIZE'] : 0;
+
+    // These don't appear to be used... are they safe to remove
     $sType0   = (isset($msg['TYPE0'])) ? $msg['TYPE0'] : 'text';
     $sType1   = (isset($msg['TYPE1'])) ? $msg['TYPE1'] : 'plain';
     if (isset($msg['INTERNALDATE'])) {
@@ -213,7 +215,7 @@ function printMessageInfo($aMsg) {
         $senderName = truncateWithEntities($senderName, $truncate_sender);
     }
 
-    $flag = $flag_end = $bold = $bold_end = $fontstr = $fontstr_end = $italic = $italic_end = '';
+    $flag = $flag_end = $fontstr = $fontstr_end = $italic = $italic_end = '';
     $bold = '<b>';
     $bold_end = '</b>';
 
@@ -492,6 +494,7 @@ function sqm_api_mailbox_select($imapConnection,$mailbox,$aConfig,$aProps) {
     /**
      * In case the properties arrays are empty set the defaults.
      */
+    // Doesn't appear to be used... safe to remove?
     $aDefaultMbxPref = array ();
 //                          MBX_PREF_SORT => 0,
 //                          MBX_PREF_LIMIT => 15,
@@ -595,7 +598,6 @@ function sqm_api_mailbox_select($imapConnection,$mailbox,$aConfig,$aProps) {
                         (!($newsort % 2) && ($newsort - 1 == $oldsort))) {
                         $aMailbox['UIDSET'][$iSetIndx] = array_reverse($aCachedMailbox['UIDSET'][$iSetIndx]);
                     } else {
-                        $server_sort_array = false;
                         $aMailbox['MSG_HEADERS'] = false;
                         $aMailbox['ID'] = false;
                     }
@@ -1142,8 +1144,6 @@ function mail_message_listing_beginning ($imapConnection,
 
     $php_self = $PHP_SELF;
 
-    $urlMailbox = urlencode($aMailbox['NAME']);
-
     if (preg_match('/^(.+)\?.+$/',$php_self,$regs)) {
         $source_url = $regs[1];
     } else {
diff --git a/functions/mime.php b/functions/mime.php
index c43b9c77..bdbff6bd 100644
--- a/functions/mime.php
+++ b/functions/mime.php
@@ -439,7 +439,6 @@ function formatAttachments($message, $exclude_id, $mailbox, $id) {
         $links['download link']['text'] = _("Download");
         $links['download link']['href'] = SM_PATH .
                 "src/download.php?absolute_dl=true&amp;passed_id=$id&amp;mailbox=$urlMailbox&amp;ent_id=$ent";
-        $ImageURL = '';
         if ($type0 =='message' && $type1 == 'rfc822') {
             $default_page = SM_PATH . 'src/read_body.php';
             $rfc822_header = $att->rfc822_header;
@@ -1125,7 +1124,6 @@ function sq_getnxtag($body, $offset){
             break;
     }
 
-    $tag_start = $pos;
     $tagname = '';
     /**
      * Look for next [\W-_], which will indicate the end of the tag name.
@@ -1185,7 +1183,6 @@ function sq_getnxtag($body, $offset){
      * At this point we loop in order to find all attributes.
      */
     $attname = '';
-    $atttype = false;
     $attary = Array();
 
     while ($pos <= strlen($body)){
@@ -1594,7 +1591,6 @@ function sq_cid2http($message, $id, $cidurl, $mailbox){
 function sq_body2div($attary, $mailbox, $message, $id){
     $me = 'sq_body2div';
     $divattary = Array('class' => "'bodyclass'");
-    $bgcolor = '#ffffff';
     $text = '#000000';
     $has_bgc_stl = $has_txt_stl = false;
     $styledef = '';
diff --git a/functions/plugin.php b/functions/plugin.php
index e0c4033a..04d7e89d 100644
--- a/functions/plugin.php
+++ b/functions/plugin.php
@@ -44,7 +44,6 @@ function use_plugin ($name) {
 function do_hook ($name) {
     global $squirrelmail_plugin_hooks, $currentHookName;
     $data = func_get_args();
-    $ret = '';
     $currentHookName = $name;
 
     if (isset($squirrelmail_plugin_hooks[$name])
diff --git a/functions/rfc822address.php b/functions/rfc822address.php
index f08ee1e8..896b37f7 100644
--- a/functions/rfc822address.php
+++ b/functions/rfc822address.php
@@ -44,7 +44,7 @@ function parseRFC822Address($sAddress,$aProps) {
     $iLimit   = $aProps['limit'];
 
     $aTokens = _getAddressTokens($sAddress);
-    $sPersonal = $sEmail = $sComment = $sGroup = '';
+    $sEmail = $sGroup = '';
     $aStack = $aComment = $aAddress = array();
     foreach ($aTokens as $sToken) {
         if ($iLimit && $iLimit == count($aAddress)) {
@@ -302,7 +302,6 @@ function _createAddressElement(&$aStack,&$aComment,&$sEmail) {
 
 function _getAddressTokens($address) {
     $aTokens = array();
-    $aAddress = array();
     $aSpecials = array('(' ,'<' ,',' ,';' ,':');
     $aReplace =  array(' (',' <',' ,',' ;',' :');
     $address = str_replace($aSpecials,$aReplace,$address);
diff --git a/functions/strings.php b/functions/strings.php
index 792c0d0f..44ad59d2 100644
--- a/functions/strings.php
+++ b/functions/strings.php
@@ -242,7 +242,6 @@ function &sqBodyWrap (&$body, $wrap) {
                    while (($mypos < $length) && ($body{$mypos} == '>')) {
                        $mypos++;
                        // skip over any spaces interleaved among the cite markers
-                       $oldpos = $mypos;
                        while (($mypos < $length) && ($body{$mypos} == ' ')) {
                            $mypos++;
                        }
diff --git a/plugins/calendar/calendar.php b/plugins/calendar/calendar.php
index 9c78338c..440c8d19 100644
--- a/plugins/calendar/calendar.php
+++ b/plugins/calendar/calendar.php
@@ -96,7 +96,7 @@ function startcalendar() {
 
 //main logic for month view of calendar
 function drawmonthview() {
-    global $year, $month, $day, $color, $calendardata, $todayis;
+    global $year, $month, $color, $calendardata, $todayis;
 
     $aday = 1 - date('w', mktime(0, 0, 0, $month, 1, $year));
     $days_in_month = date('t', mktime(0, 0, 0, $month, 1, $year));
diff --git a/plugins/mail_fetch/setup.php b/plugins/mail_fetch/setup.php
index ac1c37d1..4d4aad79 100644
--- a/plugins/mail_fetch/setup.php
+++ b/plugins/mail_fetch/setup.php
@@ -155,8 +155,6 @@
                 if ($Count == 0) {
                     $pop3->quit();
                     continue;
-                } else {
-                    $newmsgcount = $Count - $i + 1;
                 }
 
                 // Faster to get them all at once
diff --git a/plugins/newmail/setup.php b/plugins/newmail/setup.php
index f587cc1c..0e3d2c09 100644
--- a/plugins/newmail/setup.php
+++ b/plugins/newmail/setup.php
@@ -38,7 +38,6 @@
                $newmail_recent;
 
         $mailboxURL = urlencode($real_box);
-        $unseen = $recent = 0;
 
         // Skip folders for Sent and Trash
 
@@ -155,7 +154,6 @@
 
             for ($i = 0;$i < count($boxes); $i++) {
 
-                $line = '';
                 $mailbox = $boxes[$i]['formatted'];
 
                 if (! isset($boxes[$i]['unseen'])) {
diff --git a/plugins/spamcop/spamcop.php b/plugins/spamcop/spamcop.php
index b74c4127..83f57230 100644
--- a/plugins/spamcop/spamcop.php
+++ b/plugins/spamcop/spamcop.php
@@ -37,7 +37,6 @@ function getMessage_RFC822_Attachment($message, $composeMessage, $passed_id,
         $message = $message->parent;
     }
     if ($response == 'OK') {
-        $subject = encodeHeader($message->rfc822_header->subject);
         array_shift($body_a);
         $body = implode('', $body_a) . "\r\n";
                 
diff --git a/src/compose.php b/src/compose.php
index c220d343..b344b931 100644
--- a/src/compose.php
+++ b/src/compose.php
@@ -118,7 +118,6 @@ function replyAllString($header) {
     /**
      * 1) Remove the addresses we'll be sending the message 'to'
      */
-    $url_replytoall_avoid_addrs = '';
     if (isset($header->replyto)) {
         $excl_ar = $header->getAddr_a('replyto');
     }
@@ -623,7 +622,6 @@ function newMail ($mailbox='', $passed_id='', $passed_ent_id='', $action='', $se
             $orig_header = $message->rfc822_header;
         }
 
-        $encoding = $message->header->encoding;
         $type0 = $message->type0;
         $type1 = $message->type1;
         foreach ($entities as $ent) {
@@ -915,7 +913,7 @@ function getMessage_RFC822_Attachment($message, $composeMessage, $passed_id,
 function showInputForm ($session, $values=false) {
     global $send_to, $send_to_cc, $body, $startMessage, $action,
         $color, $use_signature, $signature, $prefix_sig,
-        $editor_size, $editor_height, $attachments, $subject, $newmail,
+        $editor_size, $editor_height, $subject, $newmail,
         $use_javascript_addr_book, $send_to_bcc, $passed_id, $mailbox,
         $from_htmladdr_search, $location_of_buttons, $attachment_dir,
         $username, $data_dir, $identity, $idents, $delete_draft,
diff --git a/src/left_main.php b/src/left_main.php
index dc7f4876..c561b8f5 100644
--- a/src/left_main.php
+++ b/src/left_main.php
@@ -519,7 +519,6 @@ function ListAdvancedBoxes ($boxes, $mbx, $j='ID.0000' ) {
         $pre = "<a class=\"mbx_link\" href=\"right_main.php?PG_SHOWALL=0&amp;startMessage=1&amp;mailbox=$mailboxURL\" target=\"right\">" . $pre;
         $end .= '</a>';
         if ($numMessages > 0) {
-            $urlMailbox = urlencode($mailbox);
             $end .= "\n<small>\n" .
                     '&nbsp;&nbsp;[<a class="mbx_link" href="empty_trash.php">'._("Purge").'</a>]'.
                     '</small>';
diff --git a/src/read_body.php b/src/read_body.php
index d3dd8657..6331d3d4 100644
--- a/src/read_body.php
+++ b/src/read_body.php
@@ -125,14 +125,13 @@ function ServerMDNSupport($aFlags) {
 }
 
 function SendMDN ( $mailbox, $passed_id, $sender, $message, $imapConnection) {
-    global $username, $attachment_dir, $popuser, $username,
+    global $username, $attachment_dir, $popuser, $username, $color,
            $version, $squirrelmail_language, $default_charset,
            $languages, $useSendmail, $domain, $sent_folder;
 
     sqgetGlobalVar('SERVER_NAME', $SERVER_NAME, SQ_SERVER);
 
     $header = $message->rfc822_header;
-    $hashed_attachment_dir = getHashedDir($username, $attachment_dir);
 
     $rfc822_header = new Rfc822Header();
     $content_type  = new ContentType('multipart/report');
diff --git a/src/view_header.php b/src/view_header.php
index 5948f8ac..19198b1b 100644
--- a/src/view_header.php
+++ b/src/view_header.php
@@ -27,7 +27,6 @@ require_once(SM_PATH . 'functions/url_parser.php');
 
 function parse_viewheader($imapConnection,$id, $passed_ent_id) {
 
-    $header_full = array();
     if (!$passed_ent_id) {
         $read=sqimap_run_command ($imapConnection, "FETCH $id BODY[HEADER]", 
                               true, $a, $b, TRUE);
diff --git a/themes/darkness.php b/themes/darkness.php
index 20189365..a6e11c6a 100755
--- a/themes/darkness.php
+++ b/themes/darkness.php
@@ -37,7 +37,7 @@ function IsUnique($Distance, $r, $g, $b, $usedArray)
 // Always tremble background
 // This might make people go insane.  Yes!  *Victory dance!*
 function Darkness_HeaderPlugin() {
-   global $PHP_SELF, $Darkness_Transition;
+   global $PHP_SELF;
 
    if (substr($PHP_SELF, -18) == '/src/left_main.php') {
       echo '<meta http-equiv="Page-Enter" content="' .