Skip to content

Commit 3a67ff9

Browse files
committed
Optimize code by replacing init_to_size with resize_no_init
There is no need to initialize memory with a fixed value which is overwritten in the next step. Signed-off-by: Stefan Weil <[email protected]>
1 parent bb2348b commit 3a67ff9

File tree

6 files changed

+10
-10
lines changed

6 files changed

+10
-10
lines changed

ccmain/reject.cpp

+1-1
Original file line numberDiff line numberDiff line change
@@ -232,7 +232,7 @@ float compute_reject_threshold(WERD_CHOICE* word) {
232232

233233
int blob_count = word->length();
234234
GenericVector<float> ratings;
235-
ratings.init_to_size(blob_count, 0.0f);
235+
ratings.resize_no_init(blob_count);
236236
for (int i = 0; i < blob_count; ++i) {
237237
ratings[i] = word->certainty(i);
238238
}

ccstruct/imagedata.cpp

+2-2
Original file line numberDiff line numberDiff line change
@@ -139,7 +139,7 @@ ImageData* ImageData::Build(const char* name, int page_number, const char* lang,
139139
image_data->page_number_ = page_number;
140140
image_data->language_ = lang;
141141
// Save the imagedata.
142-
image_data->image_data_.init_to_size(imagedatasize, 0);
142+
image_data->image_data_.resize_no_init(imagedatasize);
143143
memcpy(&image_data->image_data_[0], imagedata, imagedatasize);
144144
if (!image_data->AddBoxes(box_text)) {
145145
if (truth_text == NULL || truth_text[0] == '\0') {
@@ -329,7 +329,7 @@ void ImageData::SetPixInternal(Pix* pix, GenericVector<char>* image_data) {
329329
size_t size;
330330
pixWriteMem(&data, &size, pix, IFF_PNG);
331331
pixDestroy(&pix);
332-
image_data->init_to_size(size, 0);
332+
image_data->resize_no_init(size);
333333
memcpy(&(*image_data)[0], data, size);
334334
free(data);
335335
}

ccutil/serialis.cpp

+2-2
Original file line numberDiff line numberDiff line change
@@ -57,7 +57,7 @@ bool TFile::Open(const char* data, int size) {
5757
}
5858
is_writing_ = false;
5959
swap_ = false;
60-
data_->init_to_size(size, 0);
60+
data_->resize_no_init(size);
6161
memcpy(&(*data_)[0], data, size);
6262
return true;
6363
}
@@ -79,7 +79,7 @@ bool TFile::Open(FILE* fp, inT64 end_offset) {
7979
data_ = new GenericVector<char>;
8080
data_is_owned_ = true;
8181
}
82-
data_->init_to_size(size, 0);
82+
data_->resize_no_init(size);
8383
return static_cast<int>(fread(&(*data_)[0], 1, size, fp)) == size;
8484
}
8585

ccutil/tessdatamanager.cpp

+3-3
Original file line numberDiff line numberDiff line change
@@ -62,7 +62,7 @@ bool TessdataManager::LoadMemBuffer(const char *name, const char *data,
6262
fp.set_swap(swap_);
6363
if (swap_) ReverseN(&num_entries, sizeof(num_entries));
6464
GenericVector<inT64> offset_table;
65-
offset_table.init_to_size(num_entries, -1);
65+
offset_table.resize_no_init(num_entries);
6666
if (fp.FReadEndian(&offset_table[0], sizeof(offset_table[0]), num_entries) !=
6767
num_entries)
6868
return false;
@@ -72,7 +72,7 @@ bool TessdataManager::LoadMemBuffer(const char *name, const char *data,
7272
int j = i + 1;
7373
while (j < num_entries && offset_table[j] == -1) ++j;
7474
if (j < num_entries) entry_size = offset_table[j] - offset_table[i];
75-
entries_[i].init_to_size(entry_size, 0);
75+
entries_[i].resize_no_init(entry_size);
7676
if (fp.FRead(&entries_[i][0], 1, entry_size) != entry_size) return false;
7777
}
7878
}
@@ -84,7 +84,7 @@ bool TessdataManager::LoadMemBuffer(const char *name, const char *data,
8484
void TessdataManager::OverwriteEntry(TessdataType type, const char *data,
8585
int size) {
8686
is_loaded_ = true;
87-
entries_[type].init_to_size(size, 0);
87+
entries_[type].resize_no_init(size);
8888
memcpy(&entries_[type][0], data, size);
8989
}
9090

lstm/tfnetwork.cpp

+1-1
Original file line numberDiff line numberDiff line change
@@ -46,7 +46,7 @@ bool TFNetwork::Serialize(TFile* fp) const {
4646
string proto_str;
4747
model_proto_.SerializeToString(&proto_str);
4848
GenericVector<char> data;
49-
data.init_to_size(proto_str.size(), 0);
49+
data.resize_no_init(proto_str.size());
5050
memcpy(&data[0], proto_str.data(), proto_str.size());
5151
if (!data.Serialize(fp)) return false;
5252
return true;

lstm/weightmatrix.cpp

+1-1
Original file line numberDiff line numberDiff line change
@@ -150,7 +150,7 @@ bool WeightMatrix::DeSerializeOld(bool training, TFile* fp) {
150150
if (!wi_.DeSerialize(fp)) return false;
151151
GenericVector<float> old_scales;
152152
if (!old_scales.DeSerialize(fp)) return false;
153-
scales_.init_to_size(old_scales.size(), 0.0);
153+
scales_.resize_no_init(old_scales.size());
154154
for (int i = 0; i < old_scales.size(); ++i) scales_[i] = old_scales[i];
155155
} else {
156156
if (!float_array.DeSerialize(fp)) return false;

0 commit comments

Comments
 (0)