GCC compilation error

Kenny_Strawn · 07-01-2010, 10:40 PM

This is the error that I now suddenly get in this line to a class of mine (and it's the ONLY error):

Code:

librgbamap.c:11: error: expected ‘:’, ‘,’, ‘;’, ‘}’ or ‘__attribute__’ before ‘=’ token

First of all what does this error mean? Secondly, this is the line that triggered it (and didn't before):

Code:

unsigned long rgbCube = Red * Green * Blue;

There should be nothing wrong with this line. I in fact had no errors in other similar lines.

Edit: Fixed by making the class a function and changing other errors within my code as an update to my customizable GTK RGBA module. Here is the full code:

Code:

#include <glib.h>
#include <glib/gtypes.h>
#include <gtk/gtk.h>

void rgbamap(unsigned long Red, unsigned long Green, unsigned long Blue, float Alpha) {

	unsigned long rgbCube = Red * Green * Blue;
	
	double aRed = Alpha*(Red);
	double aGreen = Alpha*(Green);
	double aBlue = Alpha*(Blue);
	
	double rBlend = (1 - Alpha)*Red + Alpha*(aRed);
	double gBlend = (1 - Alpha)*Green + Alpha*(aGreen);
	double bBlend = (1 - Alpha)*Blue + Alpha*(aBlue);
        return;
};

MrCode · 07-01-2010, 10:47 PM

I assume you're talking about the code you posted in this thread? It would be better if you posted it here, so that other members won't have to keep referring to that thread to see your code...

Are you still using those nonsensical "<=" variable assignments? I really don't see the logic in that...why are you trying to compare something before it even has a value assigned to it?

I would tend to guess that's the cause of your problem.

EDIT: Thanks for posting your code.

Quote:

Reason: Fixed the error

So, GCC isn't complaining anymore? Problem solved?

Kenny_Strawn · 07-01-2010, 10:50 PM

Original post modified.

Kenny_Strawn · 07-01-2010, 11:13 PM

Quote:

Originally Posted by MrCode

So, GCC isn't complaining anymore? Problem solved?

After changing the class to a function, yes.

KenJackson · 07-01-2010, 11:32 PM

When the compiler tells me it expected a ';' on line 11, it usually means I forgot to put ';' at the end of line 10.

Sergei Steshenko · 07-01-2010, 11:35 PM

Quote:

Originally Posted by Kenny_Strawn

...
Here is the full code:

Code:

#include <glib.h>
#include <glib/gtypes.h>
#include <gtk/gtk.h>

void rgbamap(unsigned long Red, unsigned long Green, unsigned long Blue, float Alpha) {

	unsigned long rgbCube = Red * Green * Blue;
	
	double aRed = Alpha*(Red);
	double aGreen = Alpha*(Green);
	double aBlue = Alpha*(Blue);
	
	double rBlend = (1 - Alpha)*Red + Alpha*(aRed);
	double gBlend = (1 - Alpha)*Green + Alpha*(aGreen);
	double bBlend = (1 - Alpha)*Blue + Alpha*(aBlue);
        return;
};

Still senseless code. Still senselessly includes gtk+ related files and still senselessly burns CPU cycles not changing anything in the calling scope.

easuter · 07-01-2010, 11:50 PM

Quote:

Originally Posted by Sergei Steshenko

Still senseless code. Still senselessly includes gtk+ related files and still senselessly burns CPU cycles not changing anything in the calling scope.

There really is no point arguing with Mr. Strawn (eg: http://www.linuxquestions.org/questi...ml#post4021086).

Until he decides to write some scaffolding to test his code like any sane programmer does, I suspect we're going to continue to see a torrent of stupid and pointless posts flooding this sub-forum.

Kenny_Strawn · 07-02-2010, 12:07 AM

Quote:

Originally Posted by easuter

There really is no point arguing with Mr. Strawn (eg: http://www.linuxquestions.org/questi...ml#post4021086).

Until he decides to write some scaffolding to test his code like any sane programmer does, I suspect we're going to continue to see a torrent of stupid and pointless posts flooding this sub-forum.

I tried testing the function in the Linux Mint default gtkrc, and for some reason it didn't work (looked like a classic theme).

I am going to have to rewrite it or something.

Sergei Steshenko · 07-02-2010, 12:11 AM

Quote:

Originally Posted by Kenny_Strawn

...for some reason it didn't work ...

It didn't work because you completely miss fundamental concepts of programming. As a result you are writing senseless code with which you spam this forum.

zirias · 07-02-2010, 12:35 AM

Quote:

Originally Posted by Kenny_Strawn

for some reason it didn't work (looked like a classic theme).

Uhm -- lol. "Some reason" could be that your code does a handful of computations, only to throw the results completely away. Really, learn some structured/imperative programming first, start with "hello world¨...

Kenny_Strawn · 07-02-2010, 05:16 PM

Code modified so that the inclusion of GTK-related files isn't senseless:

Code:

#include <glib.h>
#include <glib/gtypes.h>
#include <gtk/gtk.h>

void rgba(guint16 Red, guint16 Green, guint16 Blue, float Alpha) {
    guint32 pixel;

    double aRed = Alpha*(Red);
    double aGreen = Alpha*(Green);
    double aBlue = Alpha*(Blue);
	
    double rBlend = (1 - Alpha)*Red + Alpha*(aRed);
    double gBlend = (1 - Alpha)*Green + Alpha*(aGreen);
    double bBlend = (1 - Alpha)*Blue + Alpha*(aBlue);
    return;
}

Note the GTK-specific guint16 and guint32 declarations and the use of the 'pixel' declaration. Now GTK-related files are included for a purpose. And the computations are formulas taken from Wikipedia for alpha blending.

zirias · 07-02-2010, 05:23 PM

Kenny_Strawn · 07-02-2010, 05:37 PM

Quote:

Originally Posted by zirias

ROFL

Reported. This is rude and an ad hominem attack. I've had enough ridicule.

zirias · 07-03-2010, 12:45 AM

Don't want to be laughed at? Don't be the clown. You had more than enough hints!

ForzaItalia2006 · 07-03-2010, 01:32 AM

Quote:

Originally Posted by zirias

Don't want to be laughed at? Don't be the clown. You had more than enough hints!

Right, hints are good and were totally appropriate in this post, because the OP seems to be missing some fundamental programming concepts.

@Kenny_Strawn: you should generally not be offended if someone tells you that you need a bit better understanding of some fundamentals. That should just help you to avoid such errors in the future.

BUT, I totally agree with Kenny_Strawn, that there are different ways to tell someone to dig further into the details. And just laughing at someone is definitely NOT the correct way. I think MOST of us were beginners at some time ;-)