C / C++ / MFC

5-Oct-10 16:03

hi,i'm currently doing a project using a software,Waspmote. I have some problems with my codings,it gave me 2 errors when i tried correcting it,

In function 'void loop()':
error: expected initializer before 'uint8_t'

In function 'void loop()':
error: a function-definition is not allowed here before '{' token

could anyone kindly help me take a look at the codings and suggest ways for me to improve on it.The code is as shown below:

int n=0;

void setup(){
// setup for Serial port over USB
USB.begin();
USB.println("USB port started...");
USB.close();

// Powering BT
BT.ON();

// Init
if(!BT.init()) USB.println("Init OK");
}
void loop()
{

// Searching for devices
if(!BT.scanNetwork()) USB.println("Scan OK");
else USB.println("Scan failed");

// Printing the devices that have been found
for(int l=0;l<BT.devices_found;l++)
{
n=0;
USB.print("---- DEVICE ");
USB.print(l,DEC);
USB.print(" ---- ");
USB.print("MAC: ");
for(int m=0;m<12;m++)
{
USB.print(BT.discovered_devices[l].mac_address[m],BYTE);
}
USB.print(" -- NAME: ");
while(BT.discovered_devices[l].name[n]!='\0')
{
USB.print(BT.discovered_devices[l].name[n],BYTE);
n++;
}
n=0;
USB.print(" -- CoD: ");
while(BT.discovered_devices[l].CoD[n]!='\0')
{
USB.print(BT.discovered_devices[l].CoD[n],BYTE);
n++;
}
USB.println("");

delay(2000);
}

// Creating a connection to a device
if(!BT.createConnection("0017E5F1CF74","02"))
{
USB.println("");
USB.print("--- CONNECTION -- MTU: ");
USB.print(BT.connection_mtu[0],BYTE);
USB.print(BT.connection_mtu[1],BYTE);
USB.println(BT.connection_mtu[2],BYTE);
}
else USB.println("Connection failed");

delay(1000);

void printData(char* data )
{
uint8_t uart=0;

printString(data,uart);
printByte('\r',uart);
printByte('\n',uart);

}
delay(2000);

return ;
}

Peter_in_27805-Oct-10 16:25

5-Oct-10 16:25

Check where you put your braces { }. It looks to me like your "printData()" function is INSIDE "loop()". Indenting your code neatly NEVER hurts and often (in cases like this) helps you see what you've written.

Software rusts. Simon Stephenson, ca 1994.

dusk855-Oct-10 17:08

5-Oct-10 17:08

thanks for the reply.I tried removing the brackets but it still doesn't work,if possible may i just trouble you to help me correct the codings because i'm rather new in programming.

Peter_in_27805-Oct-10 17:23

5-Oct-10 17:23

I didn't say remove the braces. You need to get them matching correctly, like this:

func1() {    // open 1
  ...
  ...
}            // close 1

func2() {    // open 2    
  ...
  ...
}            // close 2

What you had is more like

func1() {
  ...
  ...
  func2() {
    ...
    ...
  }

}

See how the indentation makes it easy to spot the problem?

Software rusts. Simon Stephenson, ca 1994.

dusk855-Oct-10 17:41

5-Oct-10 17:41

After looking through the codes and compiling, I still get an error:

In function 'void loop()':
error: a function-definition is not allowed here before '{' token

C++

int n=0;

void setup()
  {                              
  // setup for Serial port over USB
  USB.begin();
  USB.println("USB port started...");
  USB.close();
  
  // Powering BT
  BT.ON();
  
  // Init
  if(!BT.init()) USB.println("Init OK");
  }                              

void loop()

{
   // Searching for devices
  if(!BT.scanNetwork()) USB.println("Scan OK");
  else USB.println("Scan failed");
  
  // Printing the devices that have been found
  for(int l=0;l<BT.devices_found;l++)
  
  {
    n=0;
    USB.print("---- DEVICE ");
    USB.print(l,DEC);
    USB.print(" ----  ");
    USB.print("MAC: ");
    for(int m=0;m<12;m++)
    
    {
      USB.print(BT.discovered_devices[l].mac_address[m],BYTE);
    }
    
    USB.print(" -- NAME: ");
    while(BT.discovered_devices[l].name[n]!='\0')
    {
      USB.print(BT.discovered_devices[l].name[n],BYTE);
      n++;
    }
    
    n=0;
    USB.print(" -- CoD: ");
    while(BT.discovered_devices[l].CoD[n]!='\0')
    {
      USB.print(BT.discovered_devices[l].CoD[n],BYTE);
      n++;
    }
    USB.println("");
    
    delay(2000);
  }
  

 
  // Creating a connection to a device
  if(!BT.createConnection("0017E5F1CF74","02"))
  {
    USB.println("");
    USB.print("--- CONNECTION -- MTU: ");
    USB.print(BT.connection_mtu[0],BYTE);
    USB.print(BT.connection_mtu[1],BYTE);
    USB.println(BT.connection_mtu[2],BYTE);    
  }
  else USB.println("Connection failed");
  
  
  delay(1000);
  
  void printData(char* data )
  {
  uint8_t uart=0;
  
  printString(data,uart);
  printByte('\r',uart);
  printByte('\n',uart);

  }
 delay(2000);
   
 return  ;
}

Peter_in_27805-Oct-10 18:00

5-Oct-10 18:00

I just pasted that code into my syntax-highlighting editor, and it tells me that the very last close brace matches the one immediately after void loop(). So void printData() is still "inside" void loop(). Again, the indenting should give you a clue - void printData() isn't in column 1 like the other functions are.

Software rusts. Simon Stephenson, ca 1994.

dusk855-Oct-10 20:06

5-Oct-10 20:06

I taken

C++

void printData

out from the

C++

loop

.There are no more errors,but when i upload the codings to the waspmote IDE, it doesn't send out the text to external devices through the bluetooth device.Do you have any recommendations for me to modify my existing source code?

Peter_in_27805-Oct-10 20:17

5-Oct-10 20:17

dusk85 wrote:
Do you have any recommendations for me to modify my existing source code?
Quote Selected Text

The short answer is no. I am happy to help with getting your code to compile, but I'm not going into your bluetooth issues. I suggest you try a forum for bluetooth or waspmote (whatever that is).

Cheers,
Peter

Software rusts. Simon Stephenson, ca 1994.

MySQL 5.0 and Visual C++ 6....do they work together?

Luc Pattyn5-Oct-10 16:40

Luc Pattyn

5-Oct-10 16:40

You really should never present unformatted code. Using PRE tags would preserve indentation and make things readable, and maybe the mistake obvious.

Smile | :)

Luc Pattyn [Forum Guidelines] [Why QA sucks] [My Articles] Nil Volentibus Arduum

Please use <PRE> tags for code snippets, they preserve indentation, and improve readability.

VS2010 & SDK 7.1

Fareed Rizkalla5-Oct-10 12:43

Fareed Rizkalla

5-Oct-10 12:43

Does anyone know if I can make the SDK 7.1 directories the default instead of using the SDK 7.0! Sniff | :^)

I know that you can configure them on each solution, but it is very boring to set them on each solution.

Or should I copy and replace files of 7.0A with 7.1. Roll eyes | :rolleyes:

(Might even free up some disk space in the process)

ns5-Oct-10 7:48

5-Oct-10 7:48

Does anyone know if the new mysql libraries can be called from the old old vc6 (on windows xp)?
thanks!

Re: MySQL 5.0 and Visual C++ 6....do they work together?

_Flaviu17-Mar-13 21:09

_Flaviu

17-Mar-13 21:09

Yes, they perfectly work togheter ... swend me an email, and I'll send to you a test application ...

String

Fareed Rizkalla5-Oct-10 6:16

Fareed Rizkalla

5-Oct-10 6:16

If I assign a string to a variable.

char* t = "Hello World!";

Doesn't this string get a fixed array with a null terminator at the end?

Richard MacCutchan5-Oct-10 6:49

Richard MacCutchan

5-Oct-10 6:49

Yes. Take a look at it in the debugger if you are not sure.

Just say 'NO' to evaluated arguments for diadic functions! Ash

Fareed Rizkalla5-Oct-10 8:20

Fareed Rizkalla

5-Oct-10 8:20

wcscat_s keeps throwing an error that the string isn't null terminated! :S

Moak5-Oct-10 8:36

Moak

5-Oct-10 8:36

Are you sure it doesn't throw an exception, because the memory provided in strDestination argument is read-only?

Chat in Europe Java | [Coffee]

Now with 24% more Twitter

Richard MacCutchan5-Oct-10 9:05

Richard MacCutchan

5-Oct-10 9:05

Fareed Rizkalla wrote:
wcscat_s keeps throwing an error that the string isn't null terminated!

Yes, I'm not surprised since you are sending it a char* (i.e. ASCII string) but wcscat_s() expects Unicode.

Just say 'NO' to evaluated arguments for diadic functions! Ash

Moak5-Oct-10 6:59

Moak

5-Oct-10 6:59

No. It assigns a pointer to a fixed null-terinated string, there is a slight difference between arrays and pointers. A writable block of memory is not allocated and you should get a compiler warning because of the unsafe conversion... change the code to:

const char* t = "Hello World!";

Hope this helps Smile | :)

Chat in Europe Java | [Coffee]

Now with 24% more Twitter

Richard MacCutchan5-Oct-10 21:37

Richard MacCutchan

5-Oct-10 21:37

Moak wrote:
a pointer to a fixed null-terinated string,

Which is simply an array of characters.

Moak wrote:
there is a slight difference between arrays and pointers.

I would say there is a huge difference.

Just say 'NO' to evaluated arguments for diadic functions! Ash