can't reuse a pure c++ .h file's methods and classes

[sincnarf]

Still no luck removing the >>24 does not set the right colors, it returned to having shades of blue and black only.

[marcel]

I don't know. Can you post the updated code?
It wouldn't hurt if you did some debugging, to see what values you actually put/take out from that image.

Still no luck removing the >>24 does not set the right colors, it returned to having shades of blue and black only.

Well, things like this (meaning image handling) can't be done by guessing to add or remove something in the code. What you do has to be logic.

So I would start with debugging, see what are the pixels taken from the QImage and what are the pixels taken from the Image.

If possible, post those debug values.

Regards

[sincnarf]

Thanks again Marcel. Here's my current function and I can already load an image, though there's a problem with the colors (see attached image). The colors are somewhat "adjusted" using Format_RGB32. The original objective is just to create an exact copy from the left graphicsView to the rightgraphics view.

void MainWindow::doF1()
 {
 	QString fileName = QFileDialog::getOpenFileName(this,
                                     tr("Open Image"), QDir::currentPath());
    if (!fileName.isEmpty()) {
        QImage tempImage(fileName);
 
        if (tempImage.isNull()) {
            QMessageBox::information(this, tr("Load Warning"),
                                     tr("Cannot load %1.").arg(fileName));
            return;
        }
        QImage image = tempImage.convertToFormat(QImage::Format_RGB32);
        RGBImage rgbImage;
        rgbImage.resize(image.width(), image.height());
 
        int pixel = 0;
 		int x = 0;
 		int y = 0;
 
 		for (x= 0; x < image.width(); x++){
	 		for (y = 0; y < image.height(); y++){
	 			pixel = image.pixel(x, y);
 
				unsigned char rcomp = (unsigned char)(qRed(pixel));
				unsigned char gcomp = (unsigned char)(qGreen(pixel));
				unsigned char bcomp = (unsigned char)(qBlue(pixel));
 
				rgbImage.setPix(x, y, rcomp, gcomp, bcomp);
 
				pixel = rgbImage(x,y);
	 			image.setPixel(x, y, pixel);
 
 			}
 		}
 
 		QGraphicsScene *scene = new QGraphicsScene;
		QGraphicsPixmapItem *item = scene->addPixmap(QPixmap::fromImage(image));
 		item->setShapeMode(QGraphicsPixmapItem::BoundingRectShape);
	 	item->setFlag(QGraphicsItem::ItemIsMovable);
 		graphicsViewVis->setScene(scene);
		graphicsViewVis->show();
    }
 }

void MainWindow::doF1()
 {
 	QString fileName = QFileDialog::getOpenFileName(this,
                                     tr("Open Image"), QDir::currentPath());
    if (!fileName.isEmpty()) {
        QImage tempImage(fileName);
        
        if (tempImage.isNull()) {
            QMessageBox::information(this, tr("Load Warning"),
                                     tr("Cannot load %1.").arg(fileName));
            return;
        }
        QImage image = tempImage.convertToFormat(QImage::Format_RGB32);
        RGBImage rgbImage;
        rgbImage.resize(image.width(), image.height());
        
        int pixel = 0;
 		int x = 0;
 		int y = 0;
 		
 		for (x= 0; x < image.width(); x++){
	 		for (y = 0; y < image.height(); y++){
	 			pixel = image.pixel(x, y);
	
				unsigned char rcomp = (unsigned char)(qRed(pixel));
				unsigned char gcomp = (unsigned char)(qGreen(pixel));
				unsigned char bcomp = (unsigned char)(qBlue(pixel));

				rgbImage.setPix(x, y, rcomp, gcomp, bcomp);
				
				pixel = rgbImage(x,y);
	 			image.setPixel(x, y, pixel);
 
 			}
 		}
 		
 		QGraphicsScene *scene = new QGraphicsScene;
		QGraphicsPixmapItem *item = scene->addPixmap(QPixmap::fromImage(image));
 		item->setShapeMode(QGraphicsPixmapItem::BoundingRectShape);
	 	item->setFlag(QGraphicsItem::ItemIsMovable);
 		graphicsViewVis->setScene(scene);
		graphicsViewVis->show();
    }
 }

To copy to clipboard, switch view to plain text mode 

Observe the colors that were set in the right image (attached Format_RGB32.jpg), I tried to print the values of the rgbImage and they're exactly the same as the original/left Image, I think that there's something wrong with using QImage::Format_RGB32.

So I tried to use QImage::Indexed8 and the colors were correct , but the main problem in using QImage::Indexed8 is that the right image has white spots and the loading of the image takes too long (about 15 seconds, I thought the program hung but it didnt).

Which makes me decide to use QImage::Format_RGB32 and possibly create a way to recompute/correct the rgb values. This is my current problem.

[marcel]

You should stick to RGB32.
The problem is in line 31 of the code you posted:

pixel = rgbImage(x,y);

pixel = rgbImage(x,y);

To copy to clipboard, switch view to plain text mode 

As before, you can't assign that directly.
First convert it to qRgb, using qRed, etc...
It is the reverse transformation of what you did before.

EDIT: Actually, I think it is better to use the macros from your header to extract the components and build a qRgb:

#define RED(pixel) ((pixel) & 0xff)
#define GREEN(pixel) (((pixel)>>8) & 0xff)
#define BLUE(pixel) (((pixel)>>16) & 0xff)

#define RED(pixel) ((pixel) & 0xff)
#define GREEN(pixel) (((pixel)>>8) & 0xff)
#define BLUE(pixel) (((pixel)>>16) & 0xff)

To copy to clipboard, switch view to plain text mode 

Regards

[sincnarf]

EDIT: Actually, I think it is better to use the macros from your header to extract the components and build a qRgb:

At last! Solved the problem. Thanks !!! Thank you Marcel, expect your name in my acknowledgements page. Hehe