bus error, resize vector

[imutate]

I get a bus (or is it buss) error when I resize a vector

typedef std::vector<dou bledblvec;
n = 127;
dblvec rv;
rv.resize(n); // error happens here

Any clues ? I can post more info if required ?

[imutate]

Larry Smith wrote:

Are you checking that 'n' is not zero, and not negative?
How large might 'n' be?

OK its way past my original post
n=127;

>
If you're validating 'n' before using it, then I'd
look elswhere - something is probably trashing memory,
causing this failure as a side effect.

I looked at all the std::vectors, they all seem to be sized. This
could be tricky, what else tends to do this ? I don't have any
pointers as such except for some function pointer, but since this gets
used lots and I checked it, it is unlikely to be that.

[Daniel T.]

im*****@hotmail .co.uk wrote:

Larry Smith wrote:

>Are you checking that 'n' is not zero, and not negative?
How large might 'n' be?

OK its way past my original post
n=127;

>>
If you're validating 'n' before using it, then I'd look elswhere -
something is probably trashing memory, causing this failure as a
side effect.

I looked at all the std::vectors, they all seem to be sized. This
could be tricky, what else tends to do this ? I don't have any
pointers as such except for some function pointer, but since this gets
used lots and I checked it, it is unlikely to be that.

Here's a simple thing to do that might find the problem. Replace all
vector array references with the 'at' function. Try this:

First do a find and replace in all files from "vector" (or "std::vecto r"
if you didn't use any using directives) to "Vec".
Change all references of #include <vectorto #include "Vec.h"
Make a Vec.h file with the following in it:

#include <vector>

// taken from TC++PL 3rd Ed.

template < typename T >
class Vec : public std::vector< T {
public:
Vec():vector<T> () { }
Vec( int s ) : vector<T>(s) { }
T& operator[](int i) { retur at(i); }
const T& operator[](int i ) const { return at(i); }
};

See if that causes the error to move somewhere closer to the actual
problem.

--
There are two things that simply cannot be doubted, logic and perception.
Doubt those, and you no longer*have anyone to discuss your doubts with,
nor any ability to discuss them.

[imutate]

Daniel T. wrote:

Here's a simple thing to do that might find the problem. Replace all
vector array references with the 'at' function. Try this:

First do a find and replace in all files from "vector" (or "std::vecto r"
if you didn't use any using directives) to "Vec".
Change all references of #include <vectorto #include "Vec.h"
Make a Vec.h file with the following in it:

#include <vector>

// taken from TC++PL 3rd Ed.

template < typename T >
class Vec : public std::vector< T {
public:
Vec():vector<T> () { }
Vec( int s ) : vector<T>(s) { }
T& operator[](int i) { retur at(i); }
const T& operator[](int i ) const { return at(i); }
};

See if that causes the error to move somewhere closer to the actual
problem.

--
There are two things that simply cannot be doubted, logic and perception.
Doubt those, and you no longer have anyone to discuss your doubts with,
nor any ability to discuss them.

I replaced my matvec.h with this

#include <vector>

template < typename T >
class Vec : public std::vector< T {
public:
Vec():vector<T> () { }
Vec( int s ) : std::vector<T>( s) { }
T& operator[](int i) { return at(i); }
const T& operator[](int i ) const { return at(i); }
};

typedef Vec<doublevecdb l;
typedef Vec<vecdblmatdb l;

The compiler went "nuts" threw too many messages to print here, but I
can post them. Is there anything obviously wrong with this template
definition (typo etc) ?

[Greg Comeau]

In article <11************ **********@c28g 2000cwb.googleg roups.com>,
<im*****@hotmai l.co.ukwrote:

>
Daniel T. wrote:

>Here's a simple thing to do that might find the problem. Replace all
vector array references with the 'at' function. Try this:

First do a find and replace in all files from "vector" (or "std::vecto r"
if you didn't use any using directives) to "Vec".
Change all references of #include <vectorto #include "Vec.h"
Make a Vec.h file with the following in it:

#include <vector>

// taken from TC++PL 3rd Ed.

template < typename T >
class Vec : public std::vector< T {
public:
Vec():vector<T> () { }
Vec( int s ) : vector<T>(s) { }
T& operator[](int i) { retur at(i); }
const T& operator[](int i ) const { return at(i); }
};

See if that causes the error to move somewhere closer to the actual
problem.

--
There are two things that simply cannot be doubted, logic and perception.
Doubt those, and you no longer have anyone to discuss your doubts with,
nor any ability to discuss them.

I replaced my matvec.h with this

#include <vector>

template < typename T >
class Vec : public std::vector< T {
public:
Vec():vector<T> () { }
Vec( int s ) : std::vector<T>( s) { }
T& operator[](int i) { return at(i); }
const T& operator[](int i ) const { return at(i); }
};

typedef Vec<doublevecdb l;
typedef Vec<vecdblmatdb l;

The compiler went "nuts" threw too many messages to print here, but I
can post them. Is there anything obviously wrong with this template
definition (typo etc) ?

If that's the exact code, see:

http://www.comeaucomputing.com/techtalk/#usestd
--
Greg Comeau / 20 years of Comeauity! Intel Mac Port now in alpha!
Comeau C/C++ ONLINE == http://www.comeaucomputing.com/tryitout
World Class Compilers: Breathtaking C++, Amazing C99, Fabulous C90.
Comeau C/C++ with Dinkumware's Libraries... Have you tried it?

[imutate]

im*****@hotmail .co.uk wrote:

The compiler went "nuts" threw too many messages to print here, but I
can post them. Is there anything obviously wrong with this template
definition (typo etc) ?

OK, different errors now, I will start a new post.

[Daniel T.]

im*****@hotmail .co.uk wrote:

Daniel T. wrote:

Here's a simple thing to do that might find the problem. Replace all
vector array references with the 'at' function. Try this:

I replaced my matvec.h with this

#include <vector>

template < typename T >
class Vec : public std::vector< T {
public:
Vec():vector<T> () { }
Vec( int s ) : std::vector<T>( s) { }
T& operator[](int i) { return at(i); }
const T& operator[](int i ) const { return at(i); }
};

typedef Vec<doublevecdb l;
typedef Vec<vecdblmatdb l;

The compiler went "nuts" threw too many messages to print here, but I
can post them. Is there anything obviously wrong with this template
definition (typo etc) ?

Just post the first three errors to us and let's see what they say.

--
There are two things that simply cannot be doubted, logic and perception.
Doubt those, and you no longer*have anyone to discuss your doubts with,
nor any ability to discuss them.

[imutate]

Daniel T. wrote:

Just post the first three errors to us and let's see what they say.

Sorry, I removed this template declaration on advice from someone else,
they made a criticism. Perhaps if you could explain the context and
removed the typos it might help. Also the namespace helps too.

[imutate]

I changed it back to composition and now I get a different error
message at runtime. Doing a resize on a std::vector I get glibc
detected malloc memory corruption. The difference now is that I don't
use any arrays now, all std::vectors. Some of these are vectors of
structs.

[Daniel T.]

In article <11************ **********@m7g2 000cwm.googlegr oups.com>,
im*****@hotmail .co.uk wrote:

I changed it back to composition and now I get a different error
message at runtime. Doing a resize on a std::vector I get glibc
detected malloc memory corruption.

That's telling you that at some earlier time, you wrote to memory you
didn't have access to.

The difference now is that I don't
use any arrays now, all std::vectors. Some of these are vectors of
structs.

Now change all the op[] calls with at(). If you use all vectors, you can
do that with a global replace of '[' with "at(" and then globally
replace ']' with ")".

That will likely find your error real quick.

--
There are two things that simply cannot be doubted, logic and perception.
Doubt those, and you no longer*have anyone to discuss your doubts with,
nor any ability to discuss them.

[imutate]

Now change all the op[] calls with at(). If you use all vectors, you can

do that with a global replace of '[' with "at(" and then globally
replace ']' with ")".

Do you mean change x[i] to x(i) or something else ?