Changeset 4904


Ignore:
Timestamp:
02/17/10 22:39:53 (10 years ago)
Author:
grum
Message:

Optimize some memory leak and some bugged lines of code

Location:
extensions/AMetaData/JpegMetaData
Files:
6 added
23 edited

Legend:

Unmodified
Added
Removed
  • extensions/AMetaData/JpegMetaData/Common/Tag.class.php

    r4698 r4904  
    118118    } 
    119119 
     120    function __destruct() 
     121    { 
     122      unset($this->tagId); 
     123      unset($this->tagName); 
     124      unset($this->tagNote); 
     125      unset($this->tagValue); 
     126      unset($this->valueLabel); 
     127      unset($this->tagIsKnown); 
     128      unset($this->tagIsImplemented); 
     129      unset($this->tagIsTranslatable); 
     130    } 
     131 
    120132    /** 
    121133     * returns the Tag Id 
  • extensions/AMetaData/JpegMetaData/Common/XmlData.class.php

    r4705 r4904  
    214214      if(array_key_exists($name, $this->attributes)) 
    215215      { 
    216         $this->attributes[$name]; 
     216        $this->attributes[$name]=$value; 
    217217        return(true); 
    218218      } 
  • extensions/AMetaData/JpegMetaData/JpegMetaData.class.php

    r4698 r4904  
    168168     * 
    169169     * @Param Array $options  (optional) 
    170      * @return Array(keyName => Array('implemented' => Boolean, 'name' => String)) 
     170     * @return Array(keyName => Array( 
     171     *                            'implemented' => Boolean, 
     172     *                            'translatable' => Boolean, 
     173     *                            'name' => String)) 
    171174     */ 
    172175    static public function getTagList($options=Array()) 
     
    230233            break; 
    231234          case MAKER_PENTAX: 
     235            include_once(JPEG_METADATA_DIR."TagDefinitions/PentaxTags.class.php"); 
    232236            $tmp=new PentaxTags(); 
    233237            $schema="exif.".MAKER_PENTAX; 
     
    260264              $returned[$keyName]=Array( 
    261265                'implemented' => $tag['implemented'], 
     266                'translatable' => $tag['translatable'], 
    262267                'name' => $name 
    263268              ); 
     
    345350    public function load($file, $options = Array()) 
    346351    { 
     352      $this->unsetAll(); 
     353 
    347354      $this->initializeOptions($options); 
    348  
    349       $this->unsetAll(); 
    350355      $this->tags = Array(); 
    351356      $this->jpeg = new JpegReader($file); 
     
    530535      unset($this->tags); 
    531536      unset($this->jpeg); 
     537      unset($this->options); 
    532538    } 
    533539 
     
    536542 
    537543?> 
    538  
  • extensions/AMetaData/JpegMetaData/Readers/AppMarkerSegmentReader.class.php

    r4686 r4904  
    711711      unset($this->workData); 
    712712      unset($this->data); 
     713      unset($this->header); 
     714      unset($this->offset); 
     715      unset($this->length); 
     716      unset($this->subType); 
     717      unset($this->dataLoaded); 
    713718    } 
    714719 
  • extensions/AMetaData/JpegMetaData/Readers/CanonReader.class.php

    r4705 r4904  
    8686    } 
    8787 
     88    function __destruct() 
     89    { 
     90      parent::__destruct(); 
     91    } 
     92 
    8893 
    8994    /** 
  • extensions/AMetaData/JpegMetaData/Readers/GpsReader.class.php

    r4686 r4904  
    5454  class GpsReader extends IfdReader 
    5555  { 
     56    function __destruct() 
     57    { 
     58      parent::__destruct(); 
     59    } 
    5660 
    5761    /** 
  • extensions/AMetaData/JpegMetaData/Readers/HeightBIMReader.class.php

    r4824 r4904  
    217217        $tag=new Tag($tagId, $tagValue); 
    218218        $this->entries[]=$tag; 
     219        unset($tag); 
    219220      } 
    220221    } 
     
    224225 
    225226?> 
    226  
  • extensions/AMetaData/JpegMetaData/Readers/IfdReader.class.php

    r4825 r4904  
    157157    } 
    158158 
     159    function __destruct() 
     160    { 
     161      parent::__destruct(); 
     162    } 
     163 
    159164    /** 
    160165     * return the offset for the next IFD block 
  • extensions/AMetaData/JpegMetaData/Readers/IptcReader.class.php

    r4698 r4904  
    8787    } 
    8888 
     89    function __destruct() 
     90    { 
     91      parent::__destruct(); 
     92    } 
     93 
     94 
    8995    public function toString() 
    9096    { 
     
    132138          array_splice($this->entries, $tagT, 1); 
    133139        } 
     140        unset($tagD); 
     141        unset($tagT); 
    134142      } 
    135143    } 
     
    199207        unset($block); 
    200208      } 
     209      unset($blocks); 
    201210 
    202211      foreach($this->entries as $key => $tag) 
     
    296305          $tag=$this->tagDef->getTagById(0x0114); 
    297306          $returned=$tag['tagValues.special'][ConvertData::toUShort($values, BYTE_ORDER_BIG_ENDIAN)]; 
     307          unset($tag); 
    298308          break; 
    299309        case 0x0203: // 2:03  - Object Type Reference 
  • extensions/AMetaData/JpegMetaData/Readers/JpegReader.class.php

    r4705 r4904  
    112112      unset($this->appMarkerSegmentReader); 
    113113      unset($this->fileHandler); 
     114      unset($this->fileName); 
    114115    } 
    115116 
     
    208209          $this->appMarkerSegmentReader[]=$marker; 
    209210          $offset+=$marker->getLength(); 
     211          unset($marker); 
    210212        } 
    211213        else 
  • extensions/AMetaData/JpegMetaData/Readers/MakerNotesReader.class.php

    r4698 r4904  
    7575    protected $maker = "UNKNOWN"; 
    7676 
     77    function __destruct() 
     78    { 
     79      unset($this->maker); 
     80      parent::__destruct(); 
     81    } 
     82 
    7783    /** 
    7884     * this function return the name of the camera maker 
  • extensions/AMetaData/JpegMetaData/Readers/NikonReader.class.php

    r4705 r4904  
    9191    } 
    9292 
     93    function __destruct() 
     94    { 
     95      parent::__destruct(); 
     96    } 
    9397 
    9498    /** 
  • extensions/AMetaData/JpegMetaData/Readers/PentaxReader.class.php

    r4698 r4904  
    9191    } 
    9292 
     93    function __destruct() 
     94    { 
     95      parent::__destruct(); 
     96    } 
    9397 
    9498    /** 
     
    146150          if(array_key_exists($values[1], $tag['tagValues.special'][1])) 
    147151            $returned.=";".$tag['tagValues.special'][1][$values[1]]; 
     152          unset($tag); 
    148153          break; 
    149154        case 0x0012: // "ExposureTime", from exiftool 
  • extensions/AMetaData/JpegMetaData/Readers/TiffReader.class.php

    r4705 r4904  
    179179        $this->IFDs[]=$IFD; 
    180180        $nextIFD = $IFD->getNextIFDOffset(); 
     181        unset($IFD); 
    181182      } 
    182183    } 
  • extensions/AMetaData/JpegMetaData/Readers/XmpReader.class.php

    r4705 r4904  
    273273            $this->addTag($node->getName(), $value); 
    274274          } 
     275          unset($child); 
    275276          break; 
    276277        default: 
     
    336337 
    337338        } 
     339        unset($tagProperties); 
    338340      } 
    339341 
    340342      $this->entries[]=$tag; 
     343      unset($tag); 
    341344    } 
    342345 
     
    499502              $returned['values'][$key]=$tag['tagValues.special'][$val]; 
    500503          } 
     504          unset($tag); 
    501505          break; 
    502506        case "Iptc4xmpCore:SubjectCode": //bag 
     
    522526            if(count($tmp)>=5) 
    523527              $returned['values'][$key]['subjectDetailName']=$tmp[4]; 
     528 
     529            unset($tmp); 
    524530          } 
    525531          break; 
     
    627633          $returned=Array((int)$computed[0], (int)$computed[1]); 
    628634          $type=ByteType::URATIONAL; 
     635          unset($computed); 
    629636          break; 
    630637        /* dates & texts */ 
     
    705712 
    706713?> 
    707  
  • extensions/AMetaData/JpegMetaData/TagDefinitions/CanonTags.class.php

    r4705 r4904  
    6262    ); 
    6363 
     64    function __destruct() 
     65    { 
     66      parent::__destruct(); 
     67    } 
     68 
    6469  } // NikonTags 
    6570 
  • extensions/AMetaData/JpegMetaData/TagDefinitions/GpsTags.class.php

    r4698 r4904  
    351351    ); 
    352352 
     353 
     354    function __destruct() 
     355    { 
     356      parent::__destruct(); 
     357    } 
     358 
    353359  } 
    354360 
  • extensions/AMetaData/JpegMetaData/TagDefinitions/IfdTags.class.php

    r4698 r4904  
    15721572 
    15731573    ); 
     1574 
     1575    function __destruct() 
     1576    { 
     1577      parent::__destruct(); 
     1578    } 
    15741579  } 
    15751580 
  • extensions/AMetaData/JpegMetaData/TagDefinitions/IptcTags.class.php

    r4698 r4904  
    20952095    } 
    20962096 
     2097    function __destruct() 
     2098    { 
     2099      parent::__destruct(); 
     2100    } 
    20972101  } 
    20982102 
  • extensions/AMetaData/JpegMetaData/TagDefinitions/KnownTags.class.php

    r4698 r4904  
    103103    function __destruct() 
    104104    { 
     105      unset($this->label); 
    105106      unset($this->tags); 
    106107    } 
  • extensions/AMetaData/JpegMetaData/TagDefinitions/NikonTags.class.php

    r4705 r4904  
    6262    ); 
    6363 
     64    function __destruct() 
     65    { 
     66      parent::__destruct(); 
     67    } 
    6468  } // NikonTags 
    6569 
  • extensions/AMetaData/JpegMetaData/TagDefinitions/PentaxTags.class.php

    r4824 r4904  
    444444      ), // < MeteringMode 
    445445 
    446       // WhiteBallance, tag 0x0019 
     446      // WhiteBalance, tag 0x0019 
    447447      0x0019 => Array( 
    448         'tagName'     => "WhiteBallance", 
     448        'tagName'     => "WhiteBalance", 
    449449        'schema'      => "Pentax", 
    450450        'translatable'=> true, 
     
    467467          65535 => "User Selected" 
    468468        ) 
    469       ), // < WhiteBallance 
    470  
    471       // WhiteBallanceMode, tag 0x001a 
     469      ), // < WhiteBalance 
     470 
     471      // WhiteBalanceMode, tag 0x001a 
    472472      0x001a => Array( 
    473         'tagName'     => "WhiteBallanceMode", 
     473        'tagName'     => "WhiteBalanceMode", 
    474474        'schema'      => "Pentax", 
    475475        'translatable'=> true, 
     
    488488          0xfffe => "Preset (Fireworks?)" 
    489489        ) 
    490       ), // < < WhiteBallanceMode 
     490      ), // < < WhiteBalanceMode 
    491491 
    492492      // Saturation, tag 0x001f 
     
    16951695 
    16961696    ); 
     1697 
     1698    function __destruct() 
     1699    { 
     1700      parent::__destruct(); 
     1701    } 
    16971702  } // PentaxTags 
    16981703 
  • extensions/AMetaData/JpegMetaData/TagDefinitions/XmpTags.class.php

    r4705 r4904  
    18641864                $this->tags[$key]['implemented'] = false; 
    18651865              } 
     1866 
     1867              unset($tmpTag); 
    18661868            } 
    18671869          } 
     
    18741876    } 
    18751877 
     1878 
     1879    function __destruct() 
     1880    { 
     1881      parent::__destruct(); 
     1882    } 
    18761883  } 
    18771884 
Note: See TracChangeset for help on using the changeset viewer.