[wp-trac] [WordPress Trac] #10779: file.php: PHPDoc + optimised unzip_file()

[WordPress Trac]

#10779: file.php: PHPDoc + optimised unzip_file()
-------------------------+--------------------------------------------------
 Reporter:  dd32         |       Owner:  dd32     
     Type:  enhancement  |      Status:  new      
 Priority:  normal       |   Milestone:  2.9      
Component:  Filesystem   |     Version:  2.9      
 Severity:  normal       |    Keywords:  has-patch
-------------------------+--------------------------------------------------
Description changed by dd32:

Old description:

> Patch contains
>  * Optimised unzip_file() function
>    * Reduced the number of IO operations required (mainly is_dir())
>    * Simplifies logic to make reading easier.
>
> ---
> Also note, I played with extracting a file at a time from the ZIP, and
> writing  to the filesystem directly, While it decreased the memory usage
> from ~12MB to 3MB for  the entire unzip operation, It increased the
> operation time from 5s to 70s, So i've left it to consume memory in order
> to speed things up.

New description:

 Patch contains 2 sections:
  * Much improved PHPDoc coverage for filesystem-related functions in
 file.php
  * Optimised unzip_file() function
    * Reduced the number of IO operations required (mainly is_dir())
    * Simplifies logic to make reading easier.

 ---
 Also note, I played with extracting a file at a time from the ZIP, and
 writing  to the filesystem directly, While it decreased the memory usage
 from ~12MB to 3MB for  the entire unzip operation, It increased the
 operation time from 5s to 70s, So i've left it to consume memory in order
 to speed things up.

--

-- 
Ticket URL: <http://core.trac.wordpress.org/ticket/10779#comment:1>
WordPress Trac <http://core.trac.wordpress.org/>
WordPress blogging software