Skip to content

Commit b993543

Browse files
committed
Fix some compiler warnings (2)
1 parent 529f774 commit b993543

4 files changed

Lines changed: 11 additions & 10 deletions

File tree

libcds/include/WaveletTreeNoptrsS.h

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -89,7 +89,7 @@ namespace cds_static
8989
uint set(uint val, uint ind) const;
9090

9191
/** Recursive function for building the Wavelet Tree. */
92-
void build_level(uint **bm, uint *symbols, uint length, uint *occs);
92+
void build_level(uint **bm, uint *symbols, uint length);
9393
};
9494
};
9595
#endif

libcds/src/static/sequence/WaveletMatrix.cpp

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -24,9 +24,11 @@
2424
namespace cds_static
2525
{
2626

27-
WaveletMatrix::WaveletMatrix(const Array &symbols2, BitSequenceBuilder * bmb, Mapper * am) : Sequence(n) {
27+
WaveletMatrix::WaveletMatrix(const Array &symbols2, BitSequenceBuilder * bmb, Mapper * am) : Sequence(0) {
2828
bmb->use();
2929
n = symbols2.getLength();
30+
this->length = n; // sets Sequence::length (Sequence(0) in initializer)
31+
3032
uint *symbols = new uint[n];
3133
this->am = am;
3234
am->use();
@@ -81,8 +83,6 @@ namespace cds_static
8183
delete [] _bm;
8284
// delete [] oc;
8385
bmb->unuse();
84-
85-
this->length = n;
8686
}
8787

8888
WaveletMatrix::WaveletMatrix(uint * symbols, size_t n, BitSequenceBuilder * bmb, Mapper * am, bool deleteSymbols) : Sequence(n) {

libcds/src/static/sequence/WaveletTreeNoptrsS.cpp

Lines changed: 5 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -24,9 +24,10 @@
2424
namespace cds_static
2525
{
2626

27-
WaveletTreeNoptrsS::WaveletTreeNoptrsS(const Array &symb, BitSequenceBuilder * bmb, Mapper * am) : Sequence(n) {
27+
WaveletTreeNoptrsS::WaveletTreeNoptrsS(const Array &symb, BitSequenceBuilder * bmb, Mapper * am) : Sequence(0) {
2828
bmb->use();
2929
this->n=symb.getLength();
30+
this->length = this->n; // sets Sequence::length (Sequence(0) in initializer)
3031
this->am=am;
3132
bool deleteSymbols = true;
3233
am->use();
@@ -80,7 +81,7 @@ namespace cds_static
8081
_bm[i][j]=0;
8182
}
8283

83-
build_level(_bm,new_symb,new_n,occurrences);
84+
build_level(_bm,new_symb,new_n);
8485
bitstring = new BitSequence*[height];
8586
for(uint i=0;i<height;i++) {
8687
bitstring[i] = bmb->build(_bm[i],new_n);
@@ -148,7 +149,7 @@ namespace cds_static
148149
_bm[i][j]=0;
149150
}
150151

151-
build_level(_bm,new_symb,new_n,occurrences);
152+
build_level(_bm,new_symb,new_n);
152153
bitstring = new BitSequence*[height];
153154
for(uint i=0;i<height;i++) {
154155
bitstring[i] = bmb->build(_bm[i],new_n);
@@ -328,7 +329,7 @@ namespace cds_static
328329
return bytesBitstrings+occ->getSize()+ptrs;
329330
}
330331

331-
void WaveletTreeNoptrsS::build_level(uint **bm, uint *symbols, uint length, uint *occs) {
332+
void WaveletTreeNoptrsS::build_level(uint **bm, uint *symbols, uint length) {
332333
// for (uint i = 0; i < length; i++)
333334
// cout << " " << symbols[i];
334335
// cout << endl;

libhdt/src/libdcs/CSD_FMIndex.cpp

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -54,7 +54,7 @@ CSD_FMIndex::CSD_FMIndex(hdt::IteratorUCharString *it, bool sparse_bitsequence,
5454

5555
this->type = FMINDEX;
5656
string element;
57-
unsigned char *text;
57+
unsigned char *text; // TODO: std::vector<unsigned char>
5858
uint *bitmap = 0;
5959
// uint32_t *bitmap = 0;
6060

@@ -149,7 +149,7 @@ CSD_FMIndex::CSD_FMIndex(hdt::IteratorUCharString *it, bool sparse_bitsequence,
149149
separators = new BitSequenceRG(bitmap, len, 4);
150150
delete[] bitmap;
151151
}
152-
delete[] text;
152+
free(text);
153153
}
154154

155155
void CSD_FMIndex::build_ssa(unsigned char *text, size_t len,

0 commit comments

Comments
 (0)