added key to _get_subloc after figuring out the correct unpack magic
[dbsrgits/DBM-Deep.git] / lib / DBM / Deep / Engine.pm
index b47f58d..c7da872 100644 (file)
@@ -4,32 +4,122 @@ use strict;
 
 use Fcntl qw( :DEFAULT :flock :seek );
 
+sub precalc_sizes {
+    ##
+    # Precalculate index, bucket and bucket list sizes
+    ##
+    my $self = shift;
+
+    $self->{index_size}       = (2**8) * $self->{long_size};
+    $self->{bucket_size}      = $self->{hash_size} + $self->{long_size};
+    $self->{bucket_list_size} = $self->{max_buckets} * $self->{bucket_size};
+
+    return 1;
+}
+
+sub set_pack {
+    ##
+    # Set pack/unpack modes (see file header for more)
+    ##
+    my $self = shift;
+    my ($long_s, $long_p, $data_s, $data_p) = @_;
+
+    ##
+    # Set to 4 and 'N' for 32-bit offset tags (default).  Theoretical limit of 4
+    # GB per file.
+    #    (Perl must be compiled with largefile support for files > 2 GB)
+    #
+    # Set to 8 and 'Q' for 64-bit offsets.  Theoretical limit of 16 XB per file.
+    #    (Perl must be compiled with largefile and 64-bit long support)
+    ##
+    $self->{long_size} = $long_s ? $long_s : 4;
+    $self->{long_pack} = $long_p ? $long_p : 'N';
+
+    ##
+    # Set to 4 and 'N' for 32-bit data length prefixes.  Limit of 4 GB for each
+    # key/value. Upgrading this is possible (see above) but probably not necessary.
+    # If you need more than 4 GB for a single key or value, this module is really
+    # not for you :-)
+    ##
+    $self->{data_size} = $data_s ? $data_s : 4;
+    $self->{data_pack} = $data_p ? $data_p : 'N';
+
+    return $self->precalc_sizes();
+}
+
+sub set_digest {
+    ##
+    # Set key digest function (default is MD5)
+    ##
+    my $self = shift;
+    my ($digest_func, $hash_size) = @_;
+
+    $self->{digest} = $digest_func ? $digest_func : \&Digest::MD5::md5;
+    $self->{hash_size} = $hash_size ? $hash_size : 16;
+
+    return $self->precalc_sizes();
+}
+
+sub new {
+    my $class = shift;
+    my ($args) = @_;
+
+    my $self = bless {
+        long_size   => 4,
+        long_pack   => 'N',
+        data_size   => 4,
+        data_pack   => 'N',
+
+        digest      => \&Digest::MD5::md5,
+        hash_size   => 16,
+
+        ##
+        # Maximum number of buckets per list before another level of indexing is done.
+        # Increase this value for slightly greater speed, but larger database files.
+        # DO NOT decrease this value below 16, due to risk of recursive reindex overrun.
+        ##
+        max_buckets => 16,
+    }, $class;
+
+    $self->precalc_sizes;
+
+    return $self;
+}
+
+sub setup_fh {
+    my $self = shift;
+    my ($obj) = @_;
+
+    $self->open( $obj ) if !defined $obj->_fh;
+
+    #XXX We have to make sure we don't mess up when autoflush isn't turned on
+    unless ( $obj->_root->{inode} ) {
+        my @stats = stat($obj->_fh);
+        $obj->_root->{inode} = $stats[1];
+        $obj->_root->{end} = $stats[7];
+    }
+
+    return 1;
+}
+
 sub open {
     ##
     # Open a fh to the database, create if nonexistent.
     # Make sure file signature matches DBM::Deep spec.
     ##
     my $self = shift;
-    my $obj = shift;
-
-    if (defined($obj->_fh)) { $self->close( $obj ); }
-
-    eval {
-        local $SIG{'__DIE__'};
-        # Theoretically, adding O_BINARY should remove the need for the binmode
-        # Of course, testing it is going to be ... interesting.
-        my $flags = O_RDWR | O_CREAT | O_BINARY;
-
-        my $fh;
-        sysopen( $fh, $obj->_root->{file}, $flags )
-            or $fh = undef;
-        $obj->_root->{fh} = $fh;
-    }; if ($@ ) { $obj->_throw_error( "Received error: $@\n" ); }
-    if (! defined($obj->_fh)) {
-        return $obj->_throw_error("Cannot sysopen file: " . $obj->_root->{file} . ": $!");
-    }
+    my ($obj) = @_;
 
-    my $fh = $obj->_fh;
+    if (defined($obj->_fh)) { $self->close_fh( $obj ); }
+
+    # Theoretically, adding O_BINARY should remove the need for the binmode
+    # Of course, testing it is going to be ... interesting.
+    my $flags = O_RDWR | O_CREAT | O_BINARY;
+
+    my $fh;
+    sysopen( $fh, $obj->_root->{file}, $flags )
+        or $obj->_throw_error("Cannot sysopen file: " . $obj->_root->{file} . ": $!");
+    $obj->_root->{fh} = $fh;
 
     #XXX Can we remove this by using the right sysopen() flags?
     # Maybe ... q.v. above
@@ -52,20 +142,20 @@ sub open {
     if (!$bytes_read) {
         seek($fh, 0 + $obj->_root->{file_offset}, SEEK_SET);
         print( $fh DBM::Deep->SIG_FILE);
-        $self->create_tag($obj, $obj->_base_offset, $obj->_type, chr(0) x $DBM::Deep::INDEX_SIZE);
 
+        $self->create_tag($obj, $obj->_base_offset, $obj->_type, chr(0) x $self->{index_size});
+
+        # Why is this being printed here? I'm not seeing where anything actually points to
+        # this spot.
+        #XXX $obj->_root->{end} isn't updated from these 10 bytes that are being written
         my $plain_key = "[base]";
-        print( $fh pack($DBM::Deep::DATA_LENGTH_PACK, length($plain_key)) . $plain_key );
+        print( $fh pack($self->{data_pack}, length($plain_key)) . $plain_key );
 
         # Flush the filehandle
         my $old_fh = select $fh;
         my $old_af = $|; $| = 1; $| = $old_af;
         select $old_fh;
 
-        my @stats = stat($fh);
-        $obj->_root->{inode} = $stats[1];
-        $obj->_root->{end} = $stats[7];
-
         return 1;
     }
 
@@ -73,35 +163,29 @@ sub open {
     # Check signature was valid
     ##
     unless ($signature eq DBM::Deep->SIG_FILE) {
-        $self->close( $obj );
-        return $obj->_throw_error("Signature not found -- file is not a Deep DB");
+        $self->close_fh( $obj );
+        $obj->_throw_error("Signature not found -- file is not a Deep DB");
     }
 
-    my @stats = stat($fh);
-    $obj->_root->{inode} = $stats[1];
-    $obj->_root->{end} = $stats[7];
-
     ##
     # Get our type from master index signature
     ##
-    my $tag = $self->load_tag($obj, $obj->_base_offset);
+    my $tag = $self->load_tag($obj, $obj->_base_offset)
+        or $obj->_throw_error("Corrupted file, no master index record");
+
+    unless ($obj->{type} eq $tag->{signature}) {
+        $obj->_throw_error("File type mismatch");
+    }
 
 #XXX We probably also want to store the hash algorithm name and not assume anything
 #XXX The cool thing would be to allow a different hashing algorithm at every level
 
-    if (!$tag) {
-        return $obj->_throw_error("Corrupted file, no master index record");
-    }
-    if ($obj->{type} ne $tag->{signature}) {
-        return $obj->_throw_error("File type mismatch");
-    }
-
     return 1;
 }
 
-sub close {
+sub close_fh {
     my $self = shift;
-    my $obj = shift;
+    my ($obj) = @_;
 
     if ( my $fh = $obj->_root->{fh} ) {
         close $fh;
@@ -122,16 +206,16 @@ sub create_tag {
     my $fh = $obj->_fh;
 
     seek($fh, $offset + $obj->_root->{file_offset}, SEEK_SET);
-    print( $fh $sig . pack($DBM::Deep::DATA_LENGTH_PACK, $size) . $content );
+    print( $fh $sig . pack($self->{data_pack}, $size) . $content );
 
     if ($offset == $obj->_root->{end}) {
-        $obj->_root->{end} += DBM::Deep->SIG_SIZE + $DBM::Deep::DATA_LENGTH_SIZE + $size;
+        $obj->_root->{end} += DBM::Deep->SIG_SIZE + $self->{data_size} + $size;
     }
 
     return {
         signature => $sig,
         size => $size,
-        offset => $offset + DBM::Deep->SIG_SIZE + $DBM::Deep::DATA_LENGTH_SIZE,
+        offset => $offset + DBM::Deep->SIG_SIZE + $self->{data_size},
         content => $content
     };
 }
@@ -146,11 +230,13 @@ sub load_tag {
     my $fh = $obj->_fh;
 
     seek($fh, $offset + $obj->_root->{file_offset}, SEEK_SET);
-    if (eof $fh) { return undef; }
+
+    #XXX I'm not sure this check will work if autoflush isn't enabled ...
+    return if eof $fh;
 
     my $b;
-    read( $fh, $b, DBM::Deep->SIG_SIZE + $DBM::Deep::DATA_LENGTH_SIZE );
-    my ($sig, $size) = unpack( "A $DBM::Deep::DATA_LENGTH_PACK", $b );
+    read( $fh, $b, DBM::Deep->SIG_SIZE + $self->{data_size} );
+    my ($sig, $size) = unpack( "A $self->{data_pack}", $b );
 
     my $buffer;
     read( $fh, $buffer, $size);
@@ -158,24 +244,11 @@ sub load_tag {
     return {
         signature => $sig,
         size => $size,
-        offset => $offset + DBM::Deep->SIG_SIZE + $DBM::Deep::DATA_LENGTH_SIZE,
+        offset => $offset + DBM::Deep->SIG_SIZE + $self->{data_size},
         content => $buffer
     };
 }
 
-sub index_lookup {
-    ##
-    # Given index tag, lookup single entry in index and return .
-    ##
-    my $self = shift;
-    my ($obj, $tag, $index) = @_;
-
-    my $location = unpack($DBM::Deep::LONG_PACK, substr($tag->{content}, $index * $DBM::Deep::LONG_SIZE, $DBM::Deep::LONG_SIZE) );
-    if (!$location) { return; }
-
-    return $self->load_tag( $obj, $location );
-}
-
 sub add_bucket {
     ##
     # Adds one key/value pair to bucket list, given offset, MD5 digest of key,
@@ -183,6 +256,7 @@ sub add_bucket {
     ##
     my $self = shift;
     my ($obj, $tag, $md5, $plain_key, $value) = @_;
+
     my $keys = $tag->{content};
     my $location = 0;
     my $result = 2;
@@ -197,8 +271,10 @@ sub add_bucket {
     ##
     # Iterate through buckets, seeing if this is a new entry or a replace.
     ##
-    for (my $i=0; $i<$DBM::Deep::MAX_BUCKETS; $i++) {
-        my $subloc = unpack($DBM::Deep::LONG_PACK, substr($keys, ($i * $DBM::Deep::BUCKET_SIZE) + $DBM::Deep::HASH_SIZE, $DBM::Deep::LONG_SIZE));
+    BUCKET:
+    for (my $i = 0; $i < $self->{max_buckets}; $i++) {
+        my ($key, $subloc) = $self->_get_key_subloc( $keys, $i );
+
         if (!$subloc) {
             ##
             # Found empty bucket (end of list).  Populate and exit loop.
@@ -208,68 +284,77 @@ sub add_bucket {
             $location = $internal_ref
                 ? $value->_base_offset
                 : $root->{end};
+print "NEW: $location\n";
+
+            seek(
+                $fh,
+                $tag->{offset} + ($i * $self->{bucket_size}) + $root->{file_offset},
+                SEEK_SET,
+            );
 
-            seek($fh, $tag->{offset} + ($i * $DBM::Deep::BUCKET_SIZE) + $root->{file_offset}, SEEK_SET);
-            print( $fh $md5 . pack($DBM::Deep::LONG_PACK, $location) );
+            print( $fh $md5 . pack($self->{long_pack}, $location) );
             last;
         }
 
-        my $key = substr($keys, $i * $DBM::Deep::BUCKET_SIZE, $DBM::Deep::HASH_SIZE);
-        if ($md5 eq $key) {
-            ##
-            # Found existing bucket with same key.  Replace with new value.
-            ##
-            $result = 1;
+        if ( $md5 ne $key ) {
+            next BUCKET;
+        }
 
-            if ($internal_ref) {
-                $location = $value->_base_offset;
-                seek($fh, $tag->{offset} + ($i * $DBM::Deep::BUCKET_SIZE) + $root->{file_offset}, SEEK_SET);
-                print( $fh $md5 . pack($DBM::Deep::LONG_PACK, $location) );
-                return $result;
-            }
+        ##
+        # Found existing bucket with same key.  Replace with new value.
+        ##
+        $result = 1;
 
-            seek($fh, $subloc + DBM::Deep->SIG_SIZE + $root->{file_offset}, SEEK_SET);
-            my $size;
-            read( $fh, $size, $DBM::Deep::DATA_LENGTH_SIZE); $size = unpack($DBM::Deep::DATA_LENGTH_PACK, $size);
+        if ($internal_ref) {
+            $location = $value->_base_offset;
+            seek($fh, $tag->{offset} + ($i * $self->{bucket_size}) + $root->{file_offset}, SEEK_SET);
+            print( $fh $md5 . pack($self->{long_pack}, $location) );
+            return $result;
+        }
 
-            ##
-            # If value is a hash, array, or raw value with equal or less size, we can
-            # reuse the same content area of the database.  Otherwise, we have to create
-            # a new content area at the EOF.
-            ##
-            my $actual_length;
-            my $r = Scalar::Util::reftype( $value ) || '';
-            if ( $r eq 'HASH' || $r eq 'ARRAY' ) {
-                $actual_length = $DBM::Deep::INDEX_SIZE;
-
-                # if autobless is enabled, must also take into consideration
-                # the class name, as it is stored along with key/value.
-                if ( $root->{autobless} ) {
-                    my $value_class = Scalar::Util::blessed($value);
-                    if ( defined $value_class && !$value->isa('DBM::Deep') ) {
-                        $actual_length += length($value_class);
-                    }
-                }
-            }
-            else { $actual_length = length($value); }
+        seek($fh, $subloc + DBM::Deep->SIG_SIZE + $root->{file_offset}, SEEK_SET);
+        my $size;
+        read( $fh, $size, $self->{data_size});
+        $size = unpack($self->{data_pack}, $size);
 
-            if ($actual_length <= $size) {
-                $location = $subloc;
-            }
-            else {
-                $location = $root->{end};
-                seek($fh, $tag->{offset} + ($i * $DBM::Deep::BUCKET_SIZE) + $DBM::Deep::HASH_SIZE + $root->{file_offset}, SEEK_SET);
-                print( $fh pack($DBM::Deep::LONG_PACK, $location) );
+        ##
+        # If value is a hash, array, or raw value with equal or less size, we can
+        # reuse the same content area of the database.  Otherwise, we have to create
+        # a new content area at the EOF.
+        ##
+        my $actual_length;
+        my $r = Scalar::Util::reftype( $value ) || '';
+        if ( $r eq 'HASH' || $r eq 'ARRAY' ) {
+            $actual_length = $self->{index_size};
+
+            # if autobless is enabled, must also take into consideration
+            # the class name, as it is stored along with key/value.
+            if ( $root->{autobless} ) {
+                my $value_class = Scalar::Util::blessed($value);
+                if ( defined $value_class && !$value->isa('DBM::Deep') ) {
+                    $actual_length += length($value_class);
+                }
             }
+        }
+        else { $actual_length = length($value); }
 
-            last;
+        if ($actual_length <= $size) {
+            $location = $subloc;
+        }
+        else {
+            $location = $root->{end};
+            seek($fh, $tag->{offset} + ($i * $self->{bucket_size}) + $self->{hash_size} + $root->{file_offset}, SEEK_SET);
+            print( $fh pack($self->{long_pack}, $location) );
         }
+
+        last;
     }
 
     ##
     # If this is an internal reference, return now.
     # No need to write value or plain key
     ##
+    #XXX We need to store the key as a reference to the internal spot
     if ($internal_ref) {
         return $result;
     }
@@ -278,52 +363,12 @@ sub add_bucket {
     # If bucket didn't fit into list, split into a new index level
     ##
     if (!$location) {
-        seek($fh, $tag->{ref_loc} + $root->{file_offset}, SEEK_SET);
-        print( $fh pack($DBM::Deep::LONG_PACK, $root->{end}) );
-
-        my $index_tag = $self->create_tag($obj, $root->{end}, DBM::Deep->SIG_INDEX, chr(0) x $DBM::Deep::INDEX_SIZE);
-        my @offsets = ();
-
-        $keys .= $md5 . pack($DBM::Deep::LONG_PACK, 0);
-
-        for (my $i=0; $i<=$DBM::Deep::MAX_BUCKETS; $i++) {
-            my $key = substr($keys, $i * $DBM::Deep::BUCKET_SIZE, $DBM::Deep::HASH_SIZE);
-            if ($key) {
-                my $old_subloc = unpack($DBM::Deep::LONG_PACK, substr($keys, ($i * $DBM::Deep::BUCKET_SIZE) +
-                        $DBM::Deep::HASH_SIZE, $DBM::Deep::LONG_SIZE));
-                my $num = ord(substr($key, $tag->{ch} + 1, 1));
-
-                if ($offsets[$num]) {
-                    my $offset = $offsets[$num] + DBM::Deep->SIG_SIZE + $DBM::Deep::DATA_LENGTH_SIZE;
-                    seek($fh, $offset + $root->{file_offset}, SEEK_SET);
-                    my $subkeys;
-                    read( $fh, $subkeys, $DBM::Deep::BUCKET_LIST_SIZE);
-
-                    for (my $k=0; $k<$DBM::Deep::MAX_BUCKETS; $k++) {
-                        my $subloc = unpack($DBM::Deep::LONG_PACK, substr($subkeys, ($k * $DBM::Deep::BUCKET_SIZE) +
-                                $DBM::Deep::HASH_SIZE, $DBM::Deep::LONG_SIZE));
-                        if (!$subloc) {
-                            seek($fh, $offset + ($k * $DBM::Deep::BUCKET_SIZE) + $root->{file_offset}, SEEK_SET);
-                            print( $fh $key . pack($DBM::Deep::LONG_PACK, $old_subloc || $root->{end}) );
-                            last;
-                        }
-                    } # k loop
-                }
-                else {
-                    $offsets[$num] = $root->{end};
-                    seek($fh, $index_tag->{offset} + ($num * $DBM::Deep::LONG_SIZE) + $root->{file_offset}, SEEK_SET);
-                    print( $fh pack($DBM::Deep::LONG_PACK, $root->{end}) );
+        # re-index bucket list
 
-                    my $blist_tag = $self->create_tag($obj, $root->{end}, DBM::Deep->SIG_BLIST, chr(0) x $DBM::Deep::BUCKET_LIST_SIZE);
-
-                    seek($fh, $blist_tag->{offset} + $root->{file_offset}, SEEK_SET);
-                    print( $fh $key . pack($DBM::Deep::LONG_PACK, $old_subloc || $root->{end}) );
-                }
-            } # key is real
-        } # i loop
+        $self->split_index( $obj, $md5, $tag );
 
-        $location ||= $root->{end};
-    } # re-index bucket list
+        $location = $root->{end};
+    }
 
     ##
     # Seek to content area and store signature, value and plaintext key
@@ -338,29 +383,29 @@ sub add_bucket {
         my $r = Scalar::Util::reftype($value) || '';
         if ($r eq 'HASH') {
             print( $fh DBM::Deep->TYPE_HASH );
-            print( $fh pack($DBM::Deep::DATA_LENGTH_PACK, $DBM::Deep::INDEX_SIZE) . chr(0) x $DBM::Deep::INDEX_SIZE );
-            $content_length = $DBM::Deep::INDEX_SIZE;
+            print( $fh pack($self->{data_pack}, $self->{index_size}) . chr(0) x $self->{index_size} );
+            $content_length = $self->{index_size};
         }
         elsif ($r eq 'ARRAY') {
             print( $fh DBM::Deep->TYPE_ARRAY );
-            print( $fh pack($DBM::Deep::DATA_LENGTH_PACK, $DBM::Deep::INDEX_SIZE) . chr(0) x $DBM::Deep::INDEX_SIZE );
-            $content_length = $DBM::Deep::INDEX_SIZE;
+            print( $fh pack($self->{data_pack}, $self->{index_size}) . chr(0) x $self->{index_size} );
+            $content_length = $self->{index_size};
         }
         elsif (!defined($value)) {
             print( $fh DBM::Deep->SIG_NULL );
-            print( $fh pack($DBM::Deep::DATA_LENGTH_PACK, 0) );
+            print( $fh pack($self->{data_pack}, 0) );
             $content_length = 0;
         }
         else {
             print( $fh DBM::Deep->SIG_DATA );
-            print( $fh pack($DBM::Deep::DATA_LENGTH_PACK, length($value)) . $value );
+            print( $fh pack($self->{data_pack}, length($value)) . $value );
             $content_length = length($value);
         }
 
         ##
         # Plain key is stored AFTER value, as keys are typically fetched less often.
         ##
-        print( $fh pack($DBM::Deep::DATA_LENGTH_PACK, length($plain_key)) . $plain_key );
+        print( $fh pack($self->{data_pack}, length($plain_key)) . $plain_key );
 
         ##
         # If value is blessed, preserve class name
@@ -372,9 +417,9 @@ sub add_bucket {
                 # Blessed ref -- will restore later
                 ##
                 print( $fh chr(1) );
-                print( $fh pack($DBM::Deep::DATA_LENGTH_PACK, length($value_class)) . $value_class );
+                print( $fh pack($self->{data_pack}, length($value_class)) . $value_class );
                 $content_length += 1;
-                $content_length += $DBM::Deep::DATA_LENGTH_SIZE + length($value_class);
+                $content_length += $self->{data_size} + length($value_class);
             }
             else {
                 print( $fh chr(0) );
@@ -387,8 +432,8 @@ sub add_bucket {
         ##
         if ($location == $root->{end}) {
             $root->{end} += DBM::Deep->SIG_SIZE;
-            $root->{end} += $DBM::Deep::DATA_LENGTH_SIZE + $content_length;
-            $root->{end} += $DBM::Deep::DATA_LENGTH_SIZE + length($plain_key);
+            $root->{end} += $self->{data_size} + $content_length;
+            $root->{end} += $self->{data_size} + length($plain_key);
         }
 
         ##
@@ -421,33 +466,93 @@ sub add_bucket {
         return $result;
     }
 
-    return $obj->_throw_error("Fatal error: indexing failed -- possibly due to corruption in file");
+    $obj->_throw_error("Fatal error: indexing failed -- possibly due to corruption in file");
+}
+
+sub split_index {
+    my $self = shift;
+    my ($obj, $md5, $tag) = @_;
+
+    my $fh = $obj->_fh;
+    my $root = $obj->_root;
+    my $keys = $tag->{content};
+
+    seek($fh, $tag->{ref_loc} + $root->{file_offset}, SEEK_SET);
+    print( $fh pack($self->{long_pack}, $root->{end}) );
+
+    my $index_tag = $self->create_tag(
+        $obj,
+        $root->{end},
+        DBM::Deep->SIG_INDEX,
+        chr(0) x $self->{index_size},
+    );
+
+    my @offsets = ();
+
+    $keys .= $md5 . pack($self->{long_pack}, 0);
+
+    BUCKET:
+    for (my $i = 0; $i <= $self->{max_buckets}; $i++) {
+        my ($key, $old_subloc) = $self->_get_key_subloc( $keys, $i );
+
+        next BUCKET unless $key;
+
+        my $num = ord(substr($key, $tag->{ch} + 1, 1));
+
+        if ($offsets[$num]) {
+            my $offset = $offsets[$num] + DBM::Deep->SIG_SIZE + $self->{data_size};
+            seek($fh, $offset + $root->{file_offset}, SEEK_SET);
+            my $subkeys;
+            read( $fh, $subkeys, $self->{bucket_list_size});
+
+            for (my $k=0; $k<$self->{max_buckets}; $k++) {
+                my ($temp, $subloc) = $self->_get_key_subloc( $subkeys, $k );
+
+                if (!$subloc) {
+                    seek($fh, $offset + ($k * $self->{bucket_size}) + $root->{file_offset}, SEEK_SET);
+                    print( $fh $key . pack($self->{long_pack}, $old_subloc || $root->{end}) );
+                    last;
+                }
+            } # k loop
+        }
+        else {
+            $offsets[$num] = $root->{end};
+            seek($fh, $index_tag->{offset} + ($num * $self->{long_size}) + $root->{file_offset}, SEEK_SET);
+            print( $fh pack($self->{long_pack}, $root->{end}) );
+
+            my $blist_tag = $self->create_tag($obj, $root->{end}, DBM::Deep->SIG_BLIST, chr(0) x $self->{bucket_list_size});
+
+            seek($fh, $blist_tag->{offset} + $root->{file_offset}, SEEK_SET);
+            print( $fh $key . pack($self->{long_pack}, $old_subloc || $root->{end}) );
+        }
+    } # i loop
+
+    return;
 }
 
 sub get_bucket_value {
-       ##
-       # Fetch single value given tag and MD5 digested key.
-       ##
-       my $self = shift;
-       my ($obj, $tag, $md5) = @_;
-       my $keys = $tag->{content};
+    ##
+    # Fetch single value given tag and MD5 digested key.
+    ##
+    my $self = shift;
+    my ($obj, $tag, $md5) = @_;
+    my $keys = $tag->{content};
 
     my $fh = $obj->_fh;
 
-       ##
-       # Iterate through buckets, looking for a key match
-       ##
+    ##
+    # Iterate through buckets, looking for a key match
+    ##
     BUCKET:
-       for (my $i=0; $i<$DBM::Deep::MAX_BUCKETS; $i++) {
-               my $key = substr($keys, $i * $DBM::Deep::BUCKET_SIZE, $DBM::Deep::HASH_SIZE);
-               my $subloc = unpack($DBM::Deep::LONG_PACK, substr($keys, ($i * $DBM::Deep::BUCKET_SIZE) + $DBM::Deep::HASH_SIZE, $DBM::Deep::LONG_SIZE));
+    for (my $i = 0; $i < $self->{max_buckets}; $i++) {
+        my ($key, $subloc) = $self->_get_key_subloc( $keys, $i );
 
-               if (!$subloc) {
-                       ##
-                       # Hit end of list, no match
-                       ##
-                       return;
-               }
+        if (!$subloc) {
+            ##
+            # Hit end of list, no match
+            ##
+            return;
+        }
 
         if ( $md5 ne $key ) {
             next BUCKET;
@@ -459,7 +564,7 @@ sub get_bucket_value {
         my $signature;
         seek($fh, $subloc + $obj->_root->{file_offset}, SEEK_SET);
         read( $fh, $signature, DBM::Deep->SIG_SIZE);
-        
+
         ##
         # If value is a hash or array, return new DBM::Deep object with correct offset
         ##
@@ -469,18 +574,18 @@ sub get_bucket_value {
                 base_offset => $subloc,
                 root => $obj->_root,
             );
-            
+
             if ($obj->_root->{autobless}) {
                 ##
                 # Skip over value and plain key to see if object needs
                 # to be re-blessed
                 ##
-                seek($fh, $DBM::Deep::DATA_LENGTH_SIZE + $DBM::Deep::INDEX_SIZE, SEEK_CUR);
-                
+                seek($fh, $self->{data_size} + $self->{index_size}, SEEK_CUR);
+
                 my $size;
-                read( $fh, $size, $DBM::Deep::DATA_LENGTH_SIZE); $size = unpack($DBM::Deep::DATA_LENGTH_PACK, $size);
+                read( $fh, $size, $self->{data_size}); $size = unpack($self->{data_pack}, $size);
                 if ($size) { seek($fh, $size, SEEK_CUR); }
-                
+
                 my $bless_bit;
                 read( $fh, $bless_bit, 1);
                 if (ord($bless_bit)) {
@@ -488,59 +593,60 @@ sub get_bucket_value {
                     # Yes, object needs to be re-blessed
                     ##
                     my $class_name;
-                    read( $fh, $size, $DBM::Deep::DATA_LENGTH_SIZE); $size = unpack($DBM::Deep::DATA_LENGTH_PACK, $size);
+                    read( $fh, $size, $self->{data_size}); $size = unpack($self->{data_pack}, $size);
                     if ($size) { read( $fh, $class_name, $size); }
                     if ($class_name) { $obj = bless( $obj, $class_name ); }
                 }
             }
-            
+
             return $obj;
         }
-        
+
         ##
         # Otherwise return actual value
         ##
         elsif ($signature eq DBM::Deep->SIG_DATA) {
             my $size;
+            read( $fh, $size, $self->{data_size});
+            $size = unpack($self->{data_pack}, $size);
+
             my $value = '';
-            read( $fh, $size, $DBM::Deep::DATA_LENGTH_SIZE); $size = unpack($DBM::Deep::DATA_LENGTH_PACK, $size);
             if ($size) { read( $fh, $value, $size); }
             return $value;
         }
-        
+
         ##
         # Key exists, but content is null
         ##
         else { return; }
-       } # i loop
+    } # i loop
 
-       return;
+    return;
 }
 
 sub delete_bucket {
-       ##
-       # Delete single key/value pair given tag and MD5 digested key.
-       ##
-       my $self = shift;
-       my ($obj, $tag, $md5) = @_;
-       my $keys = $tag->{content};
+    ##
+    # Delete single key/value pair given tag and MD5 digested key.
+    ##
+    my $self = shift;
+    my ($obj, $tag, $md5) = @_;
+    my $keys = $tag->{content};
 
     my $fh = $obj->_fh;
-       
-       ##
-       # Iterate through buckets, looking for a key match
-       ##
+
+    ##
+    # Iterate through buckets, looking for a key match
+    ##
     BUCKET:
-       for (my $i=0; $i<$DBM::Deep::MAX_BUCKETS; $i++) {
-               my $key = substr($keys, $i * $DBM::Deep::BUCKET_SIZE, $DBM::Deep::HASH_SIZE);
-               my $subloc = unpack($DBM::Deep::LONG_PACK, substr($keys, ($i * $DBM::Deep::BUCKET_SIZE) + $DBM::Deep::HASH_SIZE, $DBM::Deep::LONG_SIZE));
+    for (my $i=0; $i<$self->{max_buckets}; $i++) {
+        my ($key, $subloc) = $self->_get_key_subloc( $keys, $i );
 
-               if (!$subloc) {
-                       ##
-                       # Hit end of list, no match
-                       ##
-                       return;
-               }
+        if (!$subloc) {
+            ##
+            # Hit end of list, no match
+            ##
+            return;
+        }
 
         if ( $md5 ne $key ) {
             next BUCKET;
@@ -549,38 +655,37 @@ sub delete_bucket {
         ##
         # Matched key -- delete bucket and return
         ##
-        seek($fh, $tag->{offset} + ($i * $DBM::Deep::BUCKET_SIZE) + $obj->_root->{file_offset}, SEEK_SET);
-        print( $fh substr($keys, ($i+1) * $DBM::Deep::BUCKET_SIZE ) );
-        print( $fh chr(0) x $DBM::Deep::BUCKET_SIZE );
-        
+        seek($fh, $tag->{offset} + ($i * $self->{bucket_size}) + $obj->_root->{file_offset}, SEEK_SET);
+        print( $fh substr($keys, ($i+1) * $self->{bucket_size} ) );
+        print( $fh chr(0) x $self->{bucket_size} );
+
         return 1;
-       } # i loop
+    } # i loop
 
-       return;
+    return;
 }
 
 sub bucket_exists {
-       ##
-       # Check existence of single key given tag and MD5 digested key.
-       ##
-       my $self = shift;
-       my ($obj, $tag, $md5) = @_;
-       my $keys = $tag->{content};
-       
-       ##
-       # Iterate through buckets, looking for a key match
-       ##
+    ##
+    # Check existence of single key given tag and MD5 digested key.
+    ##
+    my $self = shift;
+    my ($obj, $tag, $md5) = @_;
+    my $keys = $tag->{content};
+
+    ##
+    # Iterate through buckets, looking for a key match
+    ##
     BUCKET:
-       for (my $i=0; $i<$DBM::Deep::MAX_BUCKETS; $i++) {
-               my $key = substr($keys, $i * $DBM::Deep::BUCKET_SIZE, $DBM::Deep::HASH_SIZE);
-               my $subloc = unpack($DBM::Deep::LONG_PACK, substr($keys, ($i * $DBM::Deep::BUCKET_SIZE) + $DBM::Deep::HASH_SIZE, $DBM::Deep::LONG_SIZE));
+    for (my $i=0; $i<$self->{max_buckets}; $i++) {
+        my ($key, $subloc) = $self->_get_key_subloc( $keys, $i );
 
-               if (!$subloc) {
-                       ##
-                       # Hit end of list, no match
-                       ##
-                       return;
-               }
+        if (!$subloc) {
+            ##
+            # Hit end of list, no match
+            ##
+            return;
+        }
 
         if ( $md5 ne $key ) {
             next BUCKET;
@@ -590,138 +695,212 @@ sub bucket_exists {
         # Matched key -- return true
         ##
         return 1;
-       } # i loop
+    } # i loop
 
-       return;
+    return;
 }
 
 sub find_bucket_list {
-       ##
-       # Locate offset for bucket list, given digested key
-       ##
-       my $self = shift;
-       my ($obj, $md5) = @_;
-       
-       ##
-       # Locate offset for bucket list using digest index system
-       ##
-       my $ch = 0;
-       my $tag = $self->load_tag($obj, $obj->_base_offset);
-       if (!$tag) { return; }
-       
-       while ($tag->{signature} ne DBM::Deep->SIG_BLIST) {
-               $tag = $self->index_lookup($obj, $tag, ord(substr($md5, $ch, 1)));
-               if (!$tag) { return; }
-               $ch++;
-       }
-       
-       return $tag;
+    ##
+    # Locate offset for bucket list, given digested key
+    ##
+    my $self = shift;
+    my ($obj, $md5, $args) = @_;
+    $args = {} unless $args;
+
+    ##
+    # Locate offset for bucket list using digest index system
+    ##
+    my $tag = $self->load_tag($obj, $obj->_base_offset)
+        or $self->_throw_error( "INTERNAL ERROR - Cannot find tag" );
+#print $obj->_base_offset, " : $tag->{signature} : $tag->{offset} : $tag->{size}\n";
+
+    my $ch = 0;
+    while ($tag->{signature} ne DBM::Deep->SIG_BLIST) {
+        my $num = ord substr($md5, $ch, 1);
+
+        my $ref_loc = $tag->{offset} + ($num * $self->{long_size});
+        $tag = $self->index_lookup( $obj, $tag, $num );
+
+        if (!$tag) {
+            if ( $args->{create} ) {
+                my $fh = $obj->_fh;
+                seek($fh, $ref_loc + $obj->_root->{file_offset}, SEEK_SET);
+                print( $fh pack($self->{long_pack}, $obj->_root->{end}) );
+
+                $tag = $self->create_tag(
+                    $obj, $obj->_root->{end},
+                    DBM::Deep->SIG_BLIST,
+                    chr(0) x $self->{bucket_list_size},
+                );
+
+                $tag->{ref_loc} = $ref_loc;
+                $tag->{ch} = $ch;
+
+                last;
+            }
+            else {
+                return;
+            }
+        }
+
+        $tag->{ch} = $ch;
+        $tag->{ref_loc} = $ref_loc;
+
+        $ch++;
+    }
+
+    return $tag;
+}
+
+sub index_lookup {
+    ##
+    # Given index tag, lookup single entry in index and return .
+    ##
+    my $self = shift;
+    my ($obj, $tag, $index) = @_;
+
+    my $location = unpack(
+        $self->{long_pack},
+        substr(
+            $tag->{content},
+            $index * $self->{long_size},
+            $self->{long_size},
+        ),
+    );
+
+    if (!$location) { return; }
+
+    return $self->load_tag( $obj, $location );
 }
 
 sub traverse_index {
-       ##
-       # Scan index and recursively step into deeper levels, looking for next key.
-       ##
+    ##
+    # Scan index and recursively step into deeper levels, looking for next key.
+    ##
     my $self = shift;
     my ($obj, $offset, $ch, $force_return_next) = @_;
-    $force_return_next = undef unless $force_return_next;
-       
-       my $tag = $self->load_tag($obj, $offset );
+
+    my $tag = $self->load_tag($obj, $offset );
 
     my $fh = $obj->_fh;
-       
-       if ($tag->{signature} ne DBM::Deep->SIG_BLIST) {
-               my $content = $tag->{content};
-               my $start;
-               if ($obj->{return_next}) { $start = 0; }
-               else { $start = ord(substr($obj->{prev_md5}, $ch, 1)); }
-               
-               for (my $index = $start; $index < 256; $index++) {
-                       my $subloc = unpack($DBM::Deep::LONG_PACK, substr($content, $index * $DBM::Deep::LONG_SIZE, $DBM::Deep::LONG_SIZE) );
-                       if ($subloc) {
-                               my $result = $self->traverse_index( $obj, $subloc, $ch + 1, $force_return_next );
-                               if (defined($result)) { return $result; }
-                       }
-               } # index loop
-               
-               $obj->{return_next} = 1;
-       } # tag is an index
-       
-       elsif ($tag->{signature} eq DBM::Deep->SIG_BLIST) {
-               my $keys = $tag->{content};
-               if ($force_return_next) { $obj->{return_next} = 1; }
-               
-               ##
-               # Iterate through buckets, looking for a key match
-               ##
-               for (my $i=0; $i<$DBM::Deep::MAX_BUCKETS; $i++) {
-                       my $key = substr($keys, $i * $DBM::Deep::BUCKET_SIZE, $DBM::Deep::HASH_SIZE);
-                       my $subloc = unpack($DBM::Deep::LONG_PACK, substr($keys, ($i * $DBM::Deep::BUCKET_SIZE) + $DBM::Deep::HASH_SIZE, $DBM::Deep::LONG_SIZE));
-       
-                       if (!$subloc) {
-                               ##
-                               # End of bucket list -- return to outer loop
-                               ##
-                               $obj->{return_next} = 1;
-                               last;
-                       }
-                       elsif ($key eq $obj->{prev_md5}) {
-                               ##
-                               # Located previous key -- return next one found
-                               ##
-                               $obj->{return_next} = 1;
-                               next;
-                       }
-                       elsif ($obj->{return_next}) {
-                               ##
-                               # Seek to bucket location and skip over signature
-                               ##
-                               seek($fh, $subloc + DBM::Deep->SIG_SIZE + $obj->_root->{file_offset}, SEEK_SET);
-                               
-                               ##
-                               # Skip over value to get to plain key
-                               ##
-                               my $size;
-                               read( $fh, $size, $DBM::Deep::DATA_LENGTH_SIZE); $size = unpack($DBM::Deep::DATA_LENGTH_PACK, $size);
-                               if ($size) { seek($fh, $size, SEEK_CUR); }
-                               
-                               ##
-                               # Read in plain key and return as scalar
-                               ##
-                               my $plain_key;
-                               read( $fh, $size, $DBM::Deep::DATA_LENGTH_SIZE); $size = unpack($DBM::Deep::DATA_LENGTH_PACK, $size);
-                               if ($size) { read( $fh, $plain_key, $size); }
-                               
-                               return $plain_key;
-                       }
-               } # bucket loop
-               
-               $obj->{return_next} = 1;
-       } # tag is a bucket list
-       
-       return;
+
+    if ($tag->{signature} ne DBM::Deep->SIG_BLIST) {
+        my $content = $tag->{content};
+        my $start = $obj->{return_next} ? 0 : ord(substr($obj->{prev_md5}, $ch, 1));
+
+        for (my $index = $start; $index < 256; $index++) {
+            my $subloc = unpack(
+                $self->{long_pack},
+                substr($content, $index * $self->{long_size}, $self->{long_size}),
+            );
+
+            if ($subloc) {
+                my $result = $self->traverse_index(
+                    $obj, $subloc, $ch + 1, $force_return_next,
+                );
+
+                if (defined($result)) { return $result; }
+            }
+        } # index loop
+
+        $obj->{return_next} = 1;
+    } # tag is an index
+
+    else {
+        my $keys = $tag->{content};
+        if ($force_return_next) { $obj->{return_next} = 1; }
+
+        ##
+        # Iterate through buckets, looking for a key match
+        ##
+        for (my $i=0; $i<$self->{max_buckets}; $i++) {
+            my ($key, $subloc) = $self->_get_key_subloc( $keys, $i );
+
+            if (!$subloc) {
+                ##
+                # End of bucket list -- return to outer loop
+                ##
+                $obj->{return_next} = 1;
+                last;
+            }
+            elsif ($key eq $obj->{prev_md5}) {
+                ##
+                # Located previous key -- return next one found
+                ##
+                $obj->{return_next} = 1;
+                next;
+            }
+            elsif ($obj->{return_next}) {
+                ##
+                # Seek to bucket location and skip over signature
+                ##
+                seek($fh, $subloc + DBM::Deep->SIG_SIZE + $obj->_root->{file_offset}, SEEK_SET);
+
+                ##
+                # Skip over value to get to plain key
+                ##
+                my $size;
+                read( $fh, $size, $self->{data_size});
+                $size = unpack($self->{data_pack}, $size);
+                if ($size) { seek($fh, $size, SEEK_CUR); }
+
+                ##
+                # Read in plain key and return as scalar
+                ##
+                my $plain_key;
+                read( $fh, $size, $self->{data_size});
+                $size = unpack($self->{data_pack}, $size);
+                if ($size) { read( $fh, $plain_key, $size); }
+
+                return $plain_key;
+            }
+        } # bucket loop
+
+        $obj->{return_next} = 1;
+    } # tag is a bucket list
+
+    return;
 }
 
 sub get_next_key {
-       ##
-       # Locate next key, given digested previous one
-       ##
+    ##
+    # Locate next key, given digested previous one
+    ##
     my $self = shift;
     my ($obj) = @_;
-       
-       $obj->{prev_md5} = $_[1] ? $_[1] : undef;
-       $obj->{return_next} = 0;
-       
-       ##
-       # If the previous key was not specifed, start at the top and
-       # return the first one found.
-       ##
-       if (!$obj->{prev_md5}) {
-               $obj->{prev_md5} = chr(0) x $DBM::Deep::HASH_SIZE;
-               $obj->{return_next} = 1;
-       }
-       
-       return $self->traverse_index( $obj, $obj->_base_offset, 0 );
+
+    $obj->{prev_md5} = $_[1] ? $_[1] : undef;
+    $obj->{return_next} = 0;
+
+    ##
+    # If the previous key was not specifed, start at the top and
+    # return the first one found.
+    ##
+    if (!$obj->{prev_md5}) {
+        $obj->{prev_md5} = chr(0) x $self->{hash_size};
+        $obj->{return_next} = 1;
+    }
+
+    return $self->traverse_index( $obj, $obj->_base_offset, 0 );
+}
+
+# Utilities
+
+sub _get_key_subloc {
+    my $self = shift;
+    my ($keys, $idx) = @_;
+
+    my ($key, $subloc) = unpack(
+        "a$self->{hash_size} $self->{long_pack}",
+        substr(
+            $keys,
+            ($idx * $self->{bucket_size}),
+            $self->{bucket_size},
+        ),
+    );
+
+    return ($key, $subloc);
 }
 
 1;