isomorfeus-ferret 0.17.1 → 0.17.3

data/ext/isomorfeus_ferret_ext/frb_index.c

@@ -208,8 +208,7 @@ static VALUE frb_fis_add(VALUE self, VALUE rfi) {
  *  of the available properties.
  */
 static VALUE
-frb_fis_add_field(int argc, VALUE *argv, VALUE self)
-{
+frb_fis_add_field(int argc, VALUE *argv, VALUE self) {
     FrtFieldInfos *fis = (FrtFieldInfos *)DATA_PTR(self);
     FrtFieldInfo *fi;
     unsigned int bits = fis->bits;
@@ -233,8 +232,7 @@ frb_fis_add_field(int argc, VALUE *argv, VALUE self)
  *  Iterate through the FieldInfo objects.
  */
 static VALUE
-frb_fis_each(VALUE self)
-{
+frb_fis_each(VALUE self) {
     int i;
     FrtFieldInfos *fis = (FrtFieldInfos *)DATA_PTR(self);
 
@@ -251,8 +249,7 @@ frb_fis_each(VALUE self)
  *  Return a string representation of the FieldInfos object.
  */
 static VALUE
-frb_fis_to_s(VALUE self)
-{
+frb_fis_to_s(VALUE self) {
     FrtFieldInfos *fis = (FrtFieldInfos *)DATA_PTR(self);
     char *fis_s = frt_fis_to_s(fis);
     VALUE rfis_s = rb_str_new2(fis_s);
@@ -291,6 +288,7 @@ static VALUE frb_fis_create_index(VALUE self, VALUE rdir) {
         StringValue(rdir);
         frb_create_dir(rdir);
         store = frt_open_mdbx_store(rs2s(rdir));
+        store->create_folder(store, segm_idx_name);
         frt_index_create(store, fis);
         frt_store_close(store);
     }
@@ -306,8 +304,7 @@ static VALUE frb_fis_create_index(VALUE self, VALUE rdir) {
  *  index.
  */
 static VALUE
-frb_fis_get_fields(VALUE self)
-{
+frb_fis_get_fields(VALUE self) {
     FrtFieldInfos *fis = (FrtFieldInfos *)DATA_PTR(self);
     VALUE rfield_names = rb_ary_new();
     int i;
@@ -325,8 +322,7 @@ frb_fis_get_fields(VALUE self)
  *  in the index.
  */
 static VALUE
-frb_fis_get_tk_fields(VALUE self)
-{
+frb_fis_get_tk_fields(VALUE self) {
     FrtFieldInfos *fis = (FrtFieldInfos *)DATA_PTR(self);
     VALUE rfield_names = rb_ary_new();
     int i;
@@ -777,16 +773,13 @@ static VALUE frb_tde_to_json(int argc, VALUE *argv, VALUE self) {
     if (do_positions) {
         if (argc == 0) {
             format = "{\"document\":%d,\"frequency\":%d,\"positions\":[";
-        }
-        else {
+        } else {
             format = "[%d,%d,[";
         }
-    }
-    else {
+    } else {
         if (argc == 0) {
             format = "{\"document\":%d,\"frequency\":%d},";
-        }
-        else {
+        } else {
             format = "[%d,%d],";
         }
     }
@@ -1039,6 +1032,7 @@ static VALUE frb_iw_init(int argc, VALUE *argv, VALUE self) {
                 StringValue(rval);
                 frb_create_dir(rval);
                 store = frt_open_mdbx_store(rs2s(rval));
+                store->create_folder(store, segm_idx_name);
             }
 
             if ((rval = rb_hash_aref(roptions, sym_analyzer)) != Qnil) {
@@ -1060,8 +1054,9 @@ static VALUE frb_iw_init(int argc, VALUE *argv, VALUE self) {
         }
         if (NULL == store) {
             store = frt_open_ram_store(NULL);
+            store->create_folder(store, segm_idx_name);
         }
-        if (!create && create_if_missing && !store->exists(store, "segments")) {
+        if (!create && create_if_missing && !store->exists(store, segm_idx_name, "segments")) {
             create = true;
         }
         if (create) {
@@ -1130,8 +1125,7 @@ frb_iw_get_doc_count(VALUE self) {
 }
 
 static int
-frb_hash_to_doc_i(VALUE key, VALUE value, VALUE arg)
-{
+frb_hash_to_doc_i(VALUE key, VALUE value, VALUE arg) {
     if (key == Qundef) {
         return ST_CONTINUE;
     } else {
@@ -1151,20 +1145,18 @@ frb_hash_to_doc_i(VALUE key, VALUE value, VALUE arg)
             case T_ARRAY:
                 {
                     int i;
-                    df->destroy_data = true;
                     for (i = 0; i < RARRAY_LEN(value); i++) {
                         val = rb_obj_as_string(RARRAY_PTR(value)[i]);
-                        frt_df_add_data_len(df, rstrdup(val), RSTRING_LEN(val), rb_enc_get(val));
+                        frt_df_add_data_len(df, RSTRING_PTR(val), RSTRING_LEN(val), rb_enc_get(val));
                     }
                 }
                 break;
             case T_STRING:
-                frt_df_add_data_len(df, rs2s(value), RSTRING_LEN(value), rb_enc_get(value));
+                frt_df_add_data_len(df, RSTRING_PTR(value), RSTRING_LEN(value), rb_enc_get(value));
                 break;
             default:
                 val = rb_obj_as_string(value);
-                df->destroy_data = true;
-                frt_df_add_data_len(df, rstrdup(val), RSTRING_LEN(val), rb_enc_get(val));
+                frt_df_add_data_len(df, RSTRING_PTR(val), RSTRING_LEN(val), rb_enc_get(val));
                 break;
         }
         FRT_TRY
@@ -1181,8 +1173,7 @@ frb_hash_to_doc_i(VALUE key, VALUE value, VALUE arg)
 }
 
 static FrtDocument *
-frb_get_doc(VALUE rdoc)
-{
+frb_get_doc(VALUE rdoc) {
     VALUE val;
     FrtDocument *doc = frt_doc_new();
     FrtDocField *df;
@@ -1199,27 +1190,25 @@ frb_get_doc(VALUE rdoc)
             {
                 int i;
                 df = frt_df_new(fsym_content);
-                df->destroy_data = true;
                 for (i = 0; i < RARRAY_LEN(rdoc); i++) {
                     val = rb_obj_as_string(RARRAY_PTR(rdoc)[i]);
-                    frt_df_add_data_len(df, rstrdup(val), RSTRING_LEN(val), rb_enc_get(val));
+                    frt_df_add_data_len(df, RSTRING_PTR(val), RSTRING_LEN(val), rb_enc_get(val));
                 }
                 frt_doc_add_field(doc, df);
             }
             break;
         case T_SYMBOL:
             /* TODO: clean up this ugly cast */
-            df = frt_df_add_data(frt_df_new(fsym_content), (char *)rb_id2name(SYM2ID(rdoc)), rb_enc_get(rdoc));
+            df = frt_df_add_data(frt_df_new(fsym_content), rb_id2name(SYM2ID(rdoc)), rb_enc_get(rdoc));
             frt_doc_add_field(doc, df);
             break;
         case T_STRING:
-            df = frt_df_add_data_len(frt_df_new(fsym_content), rs2s(rdoc), RSTRING_LEN(rdoc), rb_enc_get(rdoc));
+            df = frt_df_add_data_len(frt_df_new(fsym_content), RSTRING_PTR(rdoc), RSTRING_LEN(rdoc), rb_enc_get(rdoc));
             frt_doc_add_field(doc, df);
             break;
         default:
             val = rb_obj_as_string(rdoc);
-            df = frt_df_add_data_len(frt_df_new(fsym_content), rstrdup(val), RSTRING_LEN(val), rb_enc_get(val));
-            df->destroy_data = true;
+            df = frt_df_add_data_len(frt_df_new(fsym_content), RSTRING_PTR(val), RSTRING_LEN(val), rb_enc_get(val));
             frt_doc_add_field(doc, df);
             break;
     }
@@ -1258,40 +1247,6 @@ frb_iw_add_doc(VALUE self, VALUE rdoc) {
     return self;
 }
 
-/*
- *  call-seq:
- *     iw.optimize -> iw
- *
- *  Optimize the index for searching. This commits any unwritten data to the
- *  index and optimizes the index into a single segment to improve search
- *  performance. This is an expensive operation and should not be called too
- *  often. The best time to call this is at the end of a long batch indexing
- *  process. Note that calling the optimize method do not in any way effect
- *  indexing speed (except for the time taken to complete the optimization
- *  process).
- */
-static VALUE
-frb_iw_optimize(VALUE self) {
-    int ex_code = 0;
-    const char *msg = NULL;
-
-    FrtIndexWriter *iw = (FrtIndexWriter *)DATA_PTR(self);
-
-    FRT_TRY
-        frt_iw_optimize(iw);
-    FRT_XCATCHALL
-        ex_code = xcontext.excode;
-        msg = xcontext.msg;
-        FRT_HANDLED();
-    FRT_XENDTRY
-
-    if (ex_code && msg) {
-        frb_raise(ex_code, msg);
-    }
-
-    return self;
-}
-
 /*
  *  call-seq:
  *     iw.commit -> iw
@@ -1473,8 +1428,7 @@ static VALUE frb_iw_field_infos(VALUE self) {
  *  to use the same analyzer in a QueryParser.
  */
 static VALUE
-frb_iw_get_analyzer(VALUE self)
-{
+frb_iw_get_analyzer(VALUE self) {
     FrtIndexWriter *iw = (FrtIndexWriter *)DATA_PTR(self);
     return frb_get_analyzer(iw->analyzer);
 }
@@ -1488,8 +1442,7 @@ frb_iw_get_analyzer(VALUE self)
  *  same analyzer will be used for all documents during search.
  */
 static VALUE
-frb_iw_set_analyzer(VALUE self, VALUE ranalyzer)
-{
+frb_iw_set_analyzer(VALUE self, VALUE ranalyzer) {
     FrtIndexWriter *iw = (FrtIndexWriter *)DATA_PTR(self);
 
     frt_a_deref(iw->analyzer);
@@ -1504,8 +1457,7 @@ frb_iw_set_analyzer(VALUE self, VALUE ranalyzer)
  *  Returns the current version of the index writer.
  */
 static VALUE
-frb_iw_version(VALUE self)
-{
+frb_iw_version(VALUE self) {
     FrtIndexWriter *iw = (FrtIndexWriter *)DATA_PTR(self);
     return ULL2NUM(iw->sis->version);
 }
@@ -1517,8 +1469,7 @@ frb_iw_version(VALUE self)
  *  Return the current value of chunk_size
  */
 static VALUE
-frb_iw_get_chunk_size(VALUE self)
-{
+frb_iw_get_chunk_size(VALUE self) {
     FrtIndexWriter *iw = (FrtIndexWriter *)DATA_PTR(self);
     return INT2FIX(iw->config.chunk_size);
 }
@@ -1530,8 +1481,7 @@ frb_iw_get_chunk_size(VALUE self)
  *  Set the chunk_size parameter
  */
 static VALUE
-frb_iw_set_chunk_size(VALUE self, VALUE rval)
-{
+frb_iw_set_chunk_size(VALUE self, VALUE rval) {
     FrtIndexWriter *iw = (FrtIndexWriter *)DATA_PTR(self);
     iw->config.chunk_size = FIX2INT(rval);
     return rval;
@@ -1544,8 +1494,7 @@ frb_iw_set_chunk_size(VALUE self, VALUE rval)
  *  Return the current value of max_buffer_memory
  */
 static VALUE
-frb_iw_get_max_buffer_memory(VALUE self)
-{
+frb_iw_get_max_buffer_memory(VALUE self) {
     FrtIndexWriter *iw = (FrtIndexWriter *)DATA_PTR(self);
     return INT2FIX(iw->config.max_buffer_memory);
 }
@@ -1557,8 +1506,7 @@ frb_iw_get_max_buffer_memory(VALUE self)
  *  Set the max_buffer_memory parameter
  */
 static VALUE
-frb_iw_set_max_buffer_memory(VALUE self, VALUE rval)
-{
+frb_iw_set_max_buffer_memory(VALUE self, VALUE rval) {
     FrtIndexWriter *iw = (FrtIndexWriter *)DATA_PTR(self);
     iw->config.max_buffer_memory = FIX2INT(rval);
     return rval;
@@ -1571,8 +1519,7 @@ frb_iw_set_max_buffer_memory(VALUE self, VALUE rval)
  *  Return the current value of term_index_interval
  */
 static VALUE
-frb_iw_get_index_interval(VALUE self)
-{
+frb_iw_get_index_interval(VALUE self) {
     FrtIndexWriter *iw = (FrtIndexWriter *)DATA_PTR(self);
     return INT2FIX(iw->config.index_interval);
 }
@@ -1584,8 +1531,7 @@ frb_iw_get_index_interval(VALUE self)
  *  Set the term_index_interval parameter
  */
 static VALUE
-frb_iw_set_index_interval(VALUE self, VALUE rval)
-{
+frb_iw_set_index_interval(VALUE self, VALUE rval) {
     FrtIndexWriter *iw = (FrtIndexWriter *)DATA_PTR(self);
     iw->config.index_interval = FIX2INT(rval);
     return rval;
@@ -1598,8 +1544,7 @@ frb_iw_set_index_interval(VALUE self, VALUE rval)
  *  Return the current value of doc_skip_interval
  */
 static VALUE
-frb_iw_get_skip_interval(VALUE self)
-{
+frb_iw_get_skip_interval(VALUE self) {
     FrtIndexWriter *iw = (FrtIndexWriter *)DATA_PTR(self);
     return INT2FIX(iw->config.skip_interval);
 }
@@ -1611,8 +1556,7 @@ frb_iw_get_skip_interval(VALUE self)
  *  Set the doc_skip_interval parameter
  */
 static VALUE
-frb_iw_set_skip_interval(VALUE self, VALUE rval)
-{
+frb_iw_set_skip_interval(VALUE self, VALUE rval) {
     FrtIndexWriter *iw = (FrtIndexWriter *)DATA_PTR(self);
     iw->config.skip_interval = FIX2INT(rval);
     return rval;
@@ -1625,8 +1569,7 @@ frb_iw_set_skip_interval(VALUE self, VALUE rval)
  *  Return the current value of merge_factor
  */
 static VALUE
-frb_iw_get_merge_factor(VALUE self)
-{
+frb_iw_get_merge_factor(VALUE self) {
     FrtIndexWriter *iw = (FrtIndexWriter *)DATA_PTR(self);
     return INT2FIX(iw->config.merge_factor);
 }
@@ -1638,8 +1581,7 @@ frb_iw_get_merge_factor(VALUE self)
  *  Set the merge_factor parameter
  */
 static VALUE
-frb_iw_set_merge_factor(VALUE self, VALUE rval)
-{
+frb_iw_set_merge_factor(VALUE self, VALUE rval) {
     FrtIndexWriter *iw = (FrtIndexWriter *)DATA_PTR(self);
     iw->config.merge_factor = FIX2INT(rval);
     return rval;
@@ -1652,8 +1594,7 @@ frb_iw_set_merge_factor(VALUE self, VALUE rval)
  *  Return the current value of max_buffered_docs
  */
 static VALUE
-frb_iw_get_max_buffered_docs(VALUE self)
-{
+frb_iw_get_max_buffered_docs(VALUE self) {
     FrtIndexWriter *iw = (FrtIndexWriter *)DATA_PTR(self);
     return INT2FIX(iw->config.max_buffered_docs);
 }
@@ -1665,8 +1606,7 @@ frb_iw_get_max_buffered_docs(VALUE self)
  *  Set the max_buffered_docs parameter
  */
 static VALUE
-frb_iw_set_max_buffered_docs(VALUE self, VALUE rval)
-{
+frb_iw_set_max_buffered_docs(VALUE self, VALUE rval) {
     FrtIndexWriter *iw = (FrtIndexWriter *)DATA_PTR(self);
     iw->config.max_buffered_docs = FIX2INT(rval);
     return rval;
@@ -1679,8 +1619,7 @@ frb_iw_set_max_buffered_docs(VALUE self, VALUE rval)
  *  Return the current value of max_merge_docs
  */
 static VALUE
-frb_iw_get_max_merge_docs(VALUE self)
-{
+frb_iw_get_max_merge_docs(VALUE self) {
     FrtIndexWriter *iw = (FrtIndexWriter *)DATA_PTR(self);
     return INT2FIX(iw->config.max_merge_docs);
 }
@@ -1692,8 +1631,7 @@ frb_iw_get_max_merge_docs(VALUE self)
  *  Set the max_merge_docs parameter
  */
 static VALUE
-frb_iw_set_max_merge_docs(VALUE self, VALUE rval)
-{
+frb_iw_set_max_merge_docs(VALUE self, VALUE rval) {
     FrtIndexWriter *iw = (FrtIndexWriter *)DATA_PTR(self);
     iw->config.max_merge_docs = FIX2INT(rval);
     return rval;
@@ -1706,8 +1644,7 @@ frb_iw_set_max_merge_docs(VALUE self, VALUE rval)
  *  Return the current value of max_field_length
  */
 static VALUE
-frb_iw_get_max_field_length(VALUE self)
-{
+frb_iw_get_max_field_length(VALUE self) {
     FrtIndexWriter *iw = (FrtIndexWriter *)DATA_PTR(self);
     return INT2FIX(iw->config.max_field_length);
 }
@@ -1719,8 +1656,7 @@ frb_iw_get_max_field_length(VALUE self)
  *  Set the max_field_length parameter
  */
 static VALUE
-frb_iw_set_max_field_length(VALUE self, VALUE rval)
-{
+frb_iw_set_max_field_length(VALUE self, VALUE rval) {
     FrtIndexWriter *iw = (FrtIndexWriter *)DATA_PTR(self);
     iw->config.max_field_length = FIX2INT(rval);
     return rval;
@@ -1800,6 +1736,7 @@ static VALUE frb_ir_init(VALUE self, VALUE rdir) {
                     case T_STRING:
                         frb_create_dir(rdir);
                         store = frt_open_mdbx_store(rs2s(rdir));
+                        store->create_folder(store, segm_idx_name);
                         break;
                     default:
                         FRT_RAISE(FRT_ARG_ERROR, "%s isn't a valid directory "
@@ -1821,6 +1758,7 @@ static VALUE frb_ir_init(VALUE self, VALUE rdir) {
                 case T_STRING:
                     frb_create_dir(rdir);
                     store = frt_open_mdbx_store(rs2s(rdir));
+                    store->create_folder(store, segm_idx_name);
                     break;
                 default:
                     FRT_RAISE(FRT_ARG_ERROR, "%s isn't a valid directory argument. "
@@ -1894,8 +1832,7 @@ frb_ir_set_norm(VALUE self, VALUE rdoc_id, VALUE rfield, VALUE rval) {
  *  could have null bytes.
  */
 static VALUE
-frb_ir_norms(VALUE self, VALUE rfield)
-{
+frb_ir_norms(VALUE self, VALUE rfield) {
     FrtIndexReader *ir = (FrtIndexReader *)DATA_PTR(self);
     frt_uchar *norms;
     norms = frt_ir_get_norms(ir, frb_field(rfield));
@@ -1913,8 +1850,7 @@ frb_ir_norms(VALUE self, VALUE rfield)
  *  Expert: Get the norm values into a string +buffer+ starting at +offset+.
  */
 static VALUE
-frb_ir_get_norms_into(VALUE self, VALUE rfield, VALUE rnorms, VALUE roffset)
-{
+frb_ir_get_norms_into(VALUE self, VALUE rfield, VALUE rnorms, VALUE roffset) {
     FrtIndexReader *ir = (FrtIndexReader *)DATA_PTR(self);
     int offset;
     offset = FIX2INT(roffset);
@@ -1969,8 +1905,7 @@ frb_ir_commit(VALUE self) {
  *  errors.
  */
 static VALUE
-frb_ir_close(VALUE self)
-{
+frb_ir_close(VALUE self) {
     FrtIndexReader *ir = (FrtIndexReader *)DATA_PTR(self);
     ((struct RData *)(self))->data = NULL;
     ((struct RData *)(self))->dmark = NULL;
@@ -1987,8 +1922,7 @@ frb_ir_close(VALUE self)
  *  IndexReader or committed by any other IndexReader.
  */
 static VALUE
-frb_ir_has_deletions(VALUE self)
-{
+frb_ir_has_deletions(VALUE self) {
     FrtIndexReader *ir = (FrtIndexReader *)DATA_PTR(self);
     return ir->has_deletions(ir) ? Qtrue : Qfalse;
 }
@@ -2029,8 +1963,7 @@ frb_ir_delete(VALUE self, VALUE rdoc_id) {
  *  Returns true if the document at +doc_id+ has been deleted.
  */
 static VALUE
-frb_ir_is_deleted(VALUE self, VALUE rdoc_id)
-{
+frb_ir_is_deleted(VALUE self, VALUE rdoc_id) {
     FrtIndexReader *ir = (FrtIndexReader *)DATA_PTR(self);
     return ir->is_deleted(ir, FIX2INT(rdoc_id)) ? Qtrue : Qfalse;
 }
@@ -2045,8 +1978,7 @@ frb_ir_is_deleted(VALUE self, VALUE rdoc_id)
  *  in the index.
  */
 static VALUE
-frb_ir_max_doc(VALUE self)
-{
+frb_ir_max_doc(VALUE self) {
     FrtIndexReader *ir = (FrtIndexReader *)DATA_PTR(self);
     return INT2FIX(ir->max_doc(ir));
 }
@@ -2060,8 +1992,7 @@ frb_ir_max_doc(VALUE self)
  *  deleted from the index.
  */
 static VALUE
-frb_ir_num_docs(VALUE self)
-{
+frb_ir_num_docs(VALUE self) {
     FrtIndexReader *ir = (FrtIndexReader *)DATA_PTR(self);
     return INT2FIX(ir->num_docs(ir));
 }
@@ -2076,24 +2007,36 @@ frb_ir_num_docs(VALUE self)
  *  effect on these documents.
  */
 static VALUE
-frb_ir_undelete_all(VALUE self)
-{
+frb_ir_undelete_all(VALUE self) {
     FrtIndexReader *ir = (FrtIndexReader *)DATA_PTR(self);
     frt_ir_undelete_all(ir);
     return self;
 }
 
 static VALUE
-frb_get_doc_range(FrtIndexReader *ir, int pos, int len, int max)
-{
+frb_get_doc_range(FrtIndexReader *ir, int pos, int len, int max) {
+    int ex_code = 0;
+    const char *msg = NULL;
     VALUE ary;
     int i;
     max = FRT_MIN(max, pos+len);
     len = max - pos;
     ary = rb_ary_new2(len);
-    for (i = 0; i < len; i++) {
-      rb_ary_store(ary, i, frb_get_lazy_doc(ir->get_lazy_doc(ir, i + pos)));
+
+    FRT_TRY
+        for (i = 0; i < len; i++) {
+            rb_ary_store(ary, i, frb_get_lazy_doc(ir->get_lazy_doc(ir, i + pos)));
+        }
+    FRT_XCATCHALL
+        ex_code = xcontext.excode;
+        msg = xcontext.msg;
+        FRT_HANDLED();
+    FRT_XENDTRY
+
+    if (ex_code && msg) {
+        frb_raise(ex_code, msg);
     }
+
     return ary;
 }
 
@@ -2121,7 +2064,24 @@ frb_ir_get_doc(int argc, VALUE *argv, VALUE self) {
                 rb_raise(rb_eArgError, "index %ld is out of range [%d..%ld] for "
                          "IndexReader#[]", pos, 0, max);
             }
-            return frb_get_lazy_doc(ir->get_lazy_doc(ir, pos));
+            
+            int ex_code = 0;
+            const char *msg = NULL;
+            VALUE ld = Qnil;
+            
+            FRT_TRY
+                ld = frb_get_lazy_doc(ir->get_lazy_doc(ir, pos));
+            FRT_XCATCHALL
+                ex_code = xcontext.excode;
+                msg = xcontext.msg;
+                FRT_HANDLED();
+            FRT_XENDTRY
+
+            if (ex_code && msg) {
+                frb_raise(ex_code, msg);
+            }
+
+            return ld;
         }
 
         /* check if idx is Range */
@@ -2152,8 +2112,7 @@ frb_ir_get_doc(int argc, VALUE *argv, VALUE self) {
  *  index to search the latest documents added to the index.
  */
 static VALUE
-frb_ir_is_latest(VALUE self)
-{
+frb_ir_is_latest(VALUE self) {
     FrtIndexReader *ir = (FrtIndexReader *)DATA_PTR(self);
     return frt_ir_is_latest(ir) ? Qtrue : Qfalse;
 }
@@ -2166,8 +2125,7 @@ frb_ir_is_latest(VALUE self)
  *  the index. Return nil if no such term_vector exists. See TermVector.
  */
 static VALUE
-frb_ir_term_vector(VALUE self, VALUE rdoc_id, VALUE rfield)
-{
+frb_ir_term_vector(VALUE self, VALUE rdoc_id, VALUE rfield) {
     FrtIndexReader *ir = (FrtIndexReader *)DATA_PTR(self);
     FrtTermVector *tv;
     VALUE rtv;
@@ -2176,15 +2134,13 @@ frb_ir_term_vector(VALUE self, VALUE rdoc_id, VALUE rfield)
         rtv = frb_get_tv(tv);
         frt_tv_destroy(tv);
         return rtv;
-    }
-    else {
+    } else {
         return Qnil;
     }
 }
 
 static void
-frb_add_each_tv(void *key, void *value, void *rtvs)
-{
+frb_add_each_tv(void *key, void *value, void *rtvs) {
     rb_hash_aset((VALUE)rtvs, ID2SYM((ID)key), frb_get_tv(value));
 }
 
@@ -2197,8 +2153,7 @@ frb_add_each_tv(void *key, void *value, void *rtvs)
  *  and they are referenced by field names (as symbols).
  */
 static VALUE
-frb_ir_term_vectors(VALUE self, VALUE rdoc_id)
-{
+frb_ir_term_vectors(VALUE self, VALUE rdoc_id) {
     FrtIndexReader *ir = (FrtIndexReader *)DATA_PTR(self);
     FrtHash *tvs = ir->term_vectors(ir, FIX2INT(rdoc_id));
     VALUE rtvs = rb_hash_new();
@@ -2216,8 +2171,7 @@ frb_ir_term_vectors(VALUE self, VALUE rdoc_id)
  *  See TermDocEnum for more info.
  */
 static VALUE
-frb_ir_term_docs(VALUE self)
-{
+frb_ir_term_docs(VALUE self) {
     FrtIndexReader *ir = (FrtIndexReader *)DATA_PTR(self);
     return frb_get_tde(self, ir->term_docs(ir));
 }
@@ -2230,8 +2184,7 @@ frb_ir_term_docs(VALUE self)
  *  term +term+ in the field +field+. See TermDocEnum for more info.
  */
 static VALUE
-frb_ir_term_docs_for(VALUE self, VALUE rfield, VALUE rterm)
-{
+frb_ir_term_docs_for(VALUE self, VALUE rfield, VALUE rterm) {
     FrtIndexReader *ir = (FrtIndexReader *)DATA_PTR(self);
     return frb_get_tde(self, ir_term_docs_for(ir,
                                               frb_field(rfield),
@@ -2247,8 +2200,7 @@ frb_ir_term_docs_for(VALUE self, VALUE rfield, VALUE rterm)
  *  more info.
  */
 static VALUE
-frb_ir_term_positions(VALUE self)
-{
+frb_ir_term_positions(VALUE self) {
     FrtIndexReader *ir = (FrtIndexReader *)DATA_PTR(self);
     return frb_get_tde(self, ir->term_positions(ir));
 }
@@ -2262,8 +2214,7 @@ frb_ir_term_positions(VALUE self)
  *  TermDocEnum for more info.
  */
 static VALUE
-frb_ir_t_pos_for(VALUE self, VALUE rfield, VALUE rterm)
-{
+frb_ir_t_pos_for(VALUE self, VALUE rfield, VALUE rterm) {
     FrtIndexReader *ir = (FrtIndexReader *)DATA_PTR(self);
     return frb_get_tde(self, frt_ir_term_positions_for(ir,
                                                    frb_field(rfield),
@@ -2278,8 +2229,7 @@ frb_ir_t_pos_for(VALUE self, VALUE rfield, VALUE rterm)
  *  field +field+.
  */
 static VALUE
-frb_ir_doc_freq(VALUE self, VALUE rfield, VALUE rterm)
-{
+frb_ir_doc_freq(VALUE self, VALUE rfield, VALUE rterm) {
     FrtIndexReader *ir = (FrtIndexReader *)DATA_PTR(self);
     return INT2FIX(frt_ir_doc_freq(ir,
                                frb_field(rfield),
@@ -2294,8 +2244,7 @@ frb_ir_doc_freq(VALUE self, VALUE rfield, VALUE rterm)
  *  terms in the field +field+ in the index.
  */
 static VALUE
-frb_ir_terms(VALUE self, VALUE rfield)
-{
+frb_ir_terms(VALUE self, VALUE rfield) {
     FrtIndexReader *ir = (FrtIndexReader *)DATA_PTR(self);
     return frb_get_te(self, frt_ir_terms(ir, frb_field(rfield)));
 }
@@ -2308,8 +2257,7 @@ frb_ir_terms(VALUE self, VALUE rfield)
  *  at term +term+.
  */
 static VALUE
-frb_ir_terms_from(VALUE self, VALUE rfield, VALUE rterm)
-{
+frb_ir_terms_from(VALUE self, VALUE rfield, VALUE rterm) {
     FrtIndexReader *ir = (FrtIndexReader *)DATA_PTR(self);
     return frb_get_te(self, frt_ir_terms_from(ir,
                                           frb_field(rfield),
@@ -2323,8 +2271,7 @@ frb_ir_terms_from(VALUE self, VALUE rfield, VALUE rterm)
  *  Same return a count of the number of terms in the field
  */
 static VALUE
-frb_ir_term_count(VALUE self, VALUE rfield)
-{
+frb_ir_term_count(VALUE self, VALUE rfield) {
     FrtIndexReader *ir = (FrtIndexReader *)DATA_PTR(self);
     FrtTermEnum *te = frt_ir_terms(ir, frb_field(rfield));
     int count = 0;
@@ -2345,8 +2292,7 @@ frb_ir_term_count(VALUE self, VALUE rfield)
  *  gathered from the FieldInfos object.
  */
 static VALUE
-frb_ir_fields(VALUE self)
-{
+frb_ir_fields(VALUE self) {
     FrtIndexReader *ir = (FrtIndexReader *)DATA_PTR(self);
     FrtFieldInfos *fis = ir->fis;
     VALUE rfield_names = rb_ary_new();
@@ -2379,8 +2325,7 @@ static VALUE frb_ir_field_infos(VALUE self) {
  *  of field names can also be gathered from the FieldInfos object.
  */
 static VALUE
-frb_ir_tk_fields(VALUE self)
-{
+frb_ir_tk_fields(VALUE self) {
     FrtIndexReader *ir = (FrtIndexReader *)DATA_PTR(self);
     FrtFieldInfos *fis = ir->fis;
     VALUE rfield_names = rb_ary_new();
@@ -2411,14 +2356,28 @@ static VALUE frb_ir_to_enum(VALUE self) {
 static VALUE frb_ir_each(VALUE self) {
     FrtIndexReader *ir = (FrtIndexReader *)DATA_PTR(self);
     if (rb_block_given_p()) {
+        int ex_code = 0;
+        const char *msg = NULL;
         long i;
         long max_doc = ir->max_doc(ir);
         VALUE rld;
-        for (i = 0; i < max_doc; i++) {
-            if (ir->is_deleted(ir, i)) continue;
-            rld = frb_get_lazy_doc(ir->get_lazy_doc(ir, i));
-            rb_yield(rld);
+
+        FRT_TRY
+            for (i = 0; i < max_doc; i++) {
+                if (ir->is_deleted(ir, i)) continue;
+                rld = frb_get_lazy_doc(ir->get_lazy_doc(ir, i));
+                rb_yield(rld);
+            }
+        FRT_XCATCHALL
+            ex_code = xcontext.excode;
+            msg = xcontext.msg;
+            FRT_HANDLED();
+        FRT_XENDTRY
+
+        if (ex_code && msg) {
+            frb_raise(ex_code, msg);
         }
+
         return self;
     } else {
         return frb_ir_to_enum(self);
@@ -2811,7 +2770,6 @@ void Init_IndexWriter(void) {
     rb_define_method(cIndexWriter, "close",        frb_iw_close, 0);
     rb_define_method(cIndexWriter, "add_document", frb_iw_add_doc, 1);
     rb_define_method(cIndexWriter, "<<",           frb_iw_add_doc, 1);
-    rb_define_method(cIndexWriter, "optimize",     frb_iw_optimize, 0);
     rb_define_method(cIndexWriter, "commit",       frb_iw_commit, 0);
     rb_define_method(cIndexWriter, "add_readers",  frb_iw_add_readers, 1);
     rb_define_method(cIndexWriter, "delete",       frb_iw_delete, 2);