SoloCodigo

Precisamente, si los paso por valor, ¿por qué los devuelve modificados?  

cola3 = cola1 + cola2;

es lo mismo que:
cola3.operator=(cola1.operator+(cola2));

al no estar el operador = sobrecargado nos podemos quedar con:
cola3  = cola1.operator+(cola2);

entonces vamos al operador + que termina asi:
return *this;

por lo tanto esto se reduce a (despues de la suma):
cola3 = cola1;

PERO cola1 (ahora es en realidad la variable cola del operador) es una copia TEMPORAL que desaparece al terminar el operador, por eso si no ejecutas el destructor no se destruye la memoria.

Si pones un cout << "destructor" endl; en el destructor vas a ver que se lo llama 5 veces, 2 por la copia y tres por las variables del main. Despues de llamarse al destructor por las copias lo que asigne a cola3 ya no es valido  

Si, o void qDestroy(queue_t *&queue).

El problema son las copias generadas por usar parametros por valor, asi te funcionaria:

cColaEnteros & operator + (cColaEnteros &cola);

Hoy es 17 de Octubre, dia de la Lealtad, salud compañeros.

Era el pueblo de Mayo quien sufría,
no ya el rigor de un odio forastero,
sino la vergonzosa tiranía
del olvido, la incuria y el dinero.

El mismo pueblo que ganara un día
su libertad al filo del acero
tanteaba el porvenir, y en su agonía
le hablaban sólo el Río y el Pampero.

De pronto alzó la frente y se hizo rayo
(¡era en Octubre y parecía Mayo!),
y conquistó sus nuevas primaveras.

El mismo pueblo fue y otra victoria.
Y, como ayer, enamoró a la Gloria,
¡y Juan y Eva Perón fueron banderas!

Leopoldo Marechal.

El subsuelo de la patria sublevado
Un pujante palpitar sacudía la entraña de la ciudad. Un hálito áspero crecía en densas vaharadas, mientras las multitudes iban llegando. Venían de las usinas de Puerto Nuevo, de los Talleres de Chacarita y Villa Crespo, de las manufacturas de San Martín y Vicente López, de las fundiciones de acerías del Riachuelo, de las hilanderías de Barracas... Hermanados en el mismo grito y en la misma fe, iban el peón de tambo de Cañuelas y el tornero de precisión, el fundidor el mecánico de automóviles, el tejedor, la hilandera y el peón. Era el subsuelo de la patria sublevado. Era el cimiento básico de la nación que asomaba.

Raul Scalabrini Ortiz.

-El malón peronista con protección oficial y asesoramiento policial que azotó el país, ha provocado rápidamente, por su gravedad, la exteriorización del repudio popular de todos los sectores de la República y millones de protestas. Hoy la Nación en su conjunto tiene clara conciencia del peligro que entraña el peronismo y de la urgencia de ponerle fin...En primer orden, nuestros camaradas deben organizar y organizarse para la lucha contra el peronismo hasta su aniquilamiento. Perón es el enemigo número uno del pueblo argentino. (Manifiesto publicado por el Partido Comunista, 21 de Octubre de 1945).

El 17 de Octubre.

La marcha peronista.

Dia de la Lealtad.

Esto esta mal:
free(queue);
*queue = NULL;

Si queue fue liberado entonces no accedas al mismo, queue = NULL; si es correcto.

Hola,ya he visto que os habeis avalanzado todos contra mi.

Para nada, simplemente te pedimos algo basico, ahora con el codigo en 30'' se soluciona el problema:

A ver, vamos de nuevo, a menos que sea imprescindible no queremos el programa completo (en el que estas trabajando) sino UN PROGRMA completo (para resolver este problema) que sea la MINIMA EXPRESION del problema, compilable excepto por el problema mismo.

pacorubio77: por favor reduci el problema a la minima expresion de codigo compilable (o que solo tenga el error que queres subsanar).

ahh!!

y sigo con el problema de asignarle cosas a memoria no definida XD


che, pero que mal los de C con clase al poner ese codigo.

o en otros compiladores anda?

gracias por las respuestas!!

¿Donde esta exactamente ese codigo? No deberia funcionar correctamente independientemente del compilador (si funciona es por casualidad), es mas el VC++ no te lo compila siquiera por el tema del terminador de cadena que no entra en un array de ese tamaño.

Efectivamente como ya se dijo HACE RATO: NO hace falta usar esa API para nada, con cualquier editor de recursos alcanza y sobra (ver mensaje de m0skit0).

Haciendo el casting y llamandolo asi me funciona bien:

Fijate que valor se usa como primer parametro a memcpy, deberia ser exactamente stack->pila->elementos.

stack->pila->elementos = elemens;

El valor de pila es indefinido, esta apuntando generalmente a una direccion erronea.

Si, CX (la C es de counter) es un registro de proposito general (depura tu programa y seguramente lo veas cambiar de valor) y no se garantiza ni mucho menos que mantenga su valor despues de llamar a una funcion.

getch se usa simplemente para que el programa no termine hasta que el usuario le de a enter en el teclado.