Changeset 4904 for extensions/AMetaData


Ignore:
Timestamp:
Feb 17, 2010, 10:39:53 PM (14 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.