Wrong sketch lines numbering and nonexisting errors

[sasa999]

#define MODE   1

#if MODE == 0
#include    <Servo.h>    
Servo myservo;  
#endif

With MODE = 0 it compiles without problem, but with MODE=1 it show non existing errors on non existing sketch lines. I use Arduino IDE 1.6.5.
//my sketch worked fine till last night. with Arduino IDE 1.6.4 . I don't know what happened but I coudnt compile many of my sketches. I tried change /update Arduino IDE to 1.6.5 , libraries etc, but nothing helps :(

[matthijskooijman]

I added ``` to your comment to mark the code area, so it won't mess up the less-than, greater-than :-)

Can you show the error you get? Also, could you try the latest hourly build from arduino.cc? Significant changes have been made, which I suspect might fix your problem.

[sasa999]

[img]http://s23.postimg.org/l354kfhm3/scr3.jpg[/img]
[img]http://s7.postimg.org/ro4y67kpn/scr2.jpg[/img]
It show random non exixting errors.
my sketches (all) worked fine till last night. with Arduino IDE 1.6.4 . Now I can't compile any of them with this piece of code . :(
#%$@%$#%$#
Without #if ...#endif in this fragment, it works fine too. In other places #if..#endif did not affect compilation.
Last 1.6.6 did not work with ESP8266. :( , but 1.6.4 and 1.6.5 works fine.
Have tried insert this code fragment in sample Blink for Arduino Uno - the same problem.
I use Win7 64bit

[matthijskooijman]

Did you try the latest hourly build version? That means the one in lower left of https://www.arduino.cc/en/Main/Software

You say this sketch worked fine with 1.6.4, but also with 1.6.5?

Can you also post the errors you get with the sketch you posted in the original message? You now posted a sketch without the errors, and errors without the sketch, which doesn't really help narrowing down the problem?

[sasa999]

http://s16.postimg.org/asrx3nh1x/scr_blink.jpg
For example i can take just any code.
Code is from Arduino examples - Blink.
I added

#define MODE  1

#if MODE == 0
#include <Servo.h>
Servo myservo;  
#endif

there is full code. Compiled for nano board, but on any board is errors

/*
  Blink
  Turns on an LED on for one second, then off for one second, repeatedly.

  Most Arduinos have an on-board LED you can control. On the Uno and
  Leonardo, it is attached to digital pin 13. If you're unsure what
  pin the on-board LED is connected to on your Arduino model, check
  the documentation at http://www.arduino.cc

  This example code is in the public domain.

  modified 8 May 2014
  by Scott Fitzgerald
 */
#define MODE  1

#if MODE == 0
#include <Servo.h>
Servo myservo;  
#endif

// the setup function runs once when you press reset or power the board
void setup() {
  // initialize digital pin 13 as an output.
  pinMode(13, OUTPUT);
}

// the loop function runs over and over again forever
void loop() {
  digitalWrite(13, HIGH);   // turn the LED on (HIGH is the voltage level)
  delay(1000);              // wait for a second
  digitalWrite(13, LOW);    // turn the LED off by making the voltage LOW
  delay(1000);              // wait for a second
}

In this code can help
#include <Arduino.h>
BUT in others sketches it changes nothing and errors is like without this Arduino library
without Arduino library you can see what kind of errors I got in others sketches.
Sorry I can't post my own sketches because they have about 15 different libraries included. Some of them I have no permision po publish.
There is another screenshor from my sketch - errors on line numbering that does not contain shown variables or funktions
http://s22.postimg.org/p005ca0rl/scr_7.jpg

[matthijskooijman]

In your screenshot, I see you are still using version 1.6.5, which is not the latest version. Could you please try the latest "hourly build" version (which is built from the latest bleeding edge version) to see if the issue hasn't been fixed in that version already? If you do not understand what I mean here (since I asked two times already), please also tell me, so I can explain in more detail.

[sasa999]

Tnx for help, I will try hourly build at weekend. Does it fine for ESP8266? 1.6.6 was not good at all. I need version that is good for ESP8266 too.

[matthijskooijman]

Does it fine for ESP8266? 1.6.6 was not good at all.

I don't know about that, sorry (but I also do not recall any bug reports from the ESP maintainers about things to fix on the IDE side to fix ESP support, so hopefully things have been fixed on the ESP side).

[sasa999]

Instelled Hourly Build with ESP8266 2.0.0
Board ESP-12e
got message:

Board nodemcuv2 (platform esp8266, package esp8266) is unknown Error compiling.
.
changed board to Generic ESP8266
got message:

Board generic (platform esp8266, package esp8266) is unknown Error compiling.

Nothing!!! New versions it incompatible. Will try reinstall 1.6.4 with my old libraries :(
It show errors evan at empty lines, cant see declared functions and variables . And in this errors message it show totaly wrong line numbering!

F:\ESP8266\v3_1.05\v3_1.05.ino: In function 'void loop()':
v3_1.05:326: error: 'resetControls' was not declared in this scope
  if (strstr(packetBuffer, "/1/push2")) resetControls();
v3_1.05:327: error: 'fullResetControls' was not declared in this scope
  if (strstr(packetBuffer, "/2/push1")) fullResetControls();
v3_1.05:330: error: 'calculate' was not declared in this scope
  float val = calculate(packetSize);
v3_1.05:334: error: 'resetFader' was not declared in this scope
  resetFader("/1/fader1");
v3_1.05:341: error: 'calculate' was not declared in this scope
  float val2 = calculate(packetSize);
v3_1.05:344: error: 'resetFader' was not declared in this scope
  resetFader("/1/fader2");
v3_1.05:351: error: 'calculate' was not declared in this scope
  if (strstr(packetBuffer, "fad"))  fadder[int(packetBuffer[8]) - 48] = calculate(packetSize);
v3_1.05:352: error: 'calculate' was not declared in this scope
  if (strstr(packetBuffer, "tog"))  toggle[int(packetBuffer[9]) - 48] = int(calculate(packetSize));
v3_1.05:353: error: 'calculate' was not declared in this scope
  if (strstr(packetBuffer, "push"))  push[int(packetBuffer[7]) - 48] = int(calculate(packetSize));
v3_1.05:361: error: 'PageOne' was not declared in this scope
  if (page == 1) PageOne();
v3_1.05:468: error: 'readControlParameters' was not declared in this scope
  readControlParameters();
v3_1.05:471: error: 'StartedActions' was not declared in this scope
  StartedActions();
F:\ESP8266\v3_1.05\v3_1.05.ino: In function 'void resetControls()':
F:\ESP8266\v3_1.05\v3_1.05.ino:495:31: warning: deprecated conversion from string constant to 'char*' [-Wwrite-strings]
  sendMessage("/1/fader1", 0.0);
F:\ESP8266\v3_1.05\v3_1.05.ino:497:31: warning: deprecated conversion from string constant to 'char*' [-Wwrite-strings]
  sendMessage("/1/fader2", 0.0);
F:\ESP8266\v3_1.05\v3_1.05.ino: In function 'void fullResetControls()':
F:\ESP8266\v3_1.05\v3_1.05.ino:502:31: warning: deprecated conversion from string constant to 'char*' [-Wwrite-strings]
  sendMessage("/1/fader1", 0.0);
F:\ESP8266\v3_1.05\v3_1.05.ino:504:31: warning: deprecated conversion from string constant to 'char*' [-Wwrite-strings]
  sendMessage("/1/fader2", 0.0);
F:\ESP8266\v3_1.05\v3_1.05.ino:506:32: warning: deprecated conversion from string constant to 'char*' [-Wwrite-strings]
  sendMessage("/1/toggle1", 0.0);
F:\ESP8266\v3_1.05\v3_1.05.ino:507:32: warning: deprecated conversion from string constant to 'char*' [-Wwrite-strings]
  sendMessage("/1/toggle2", 0.0);
F:\ESP8266\v3_1.05\v3_1.05.ino:508:31: warning: deprecated conversion from string constant to 'char*' [-Wwrite-strings]
  sendMessage("/2/fader1", inc);
F:\ESP8266\v3_1.05\v3_1.05.ino:509:31: warning: deprecated conversion from string constant to 'char*' [-Wwrite-strings]
  sendMessage("/2/fader2", inc);
F:\ESP8266\v3_1.05\v3_1.05.ino:510:31: warning: deprecated conversion from string constant to 'char*' [-Wwrite-strings]
  sendMessage("/2/fader3", inc);
F:\ESP8266\v3_1.05\v3_1.05.ino:511:31: warning: deprecated conversion from string constant to 'char*' [-Wwrite-strings]
  sendMessage("/2/fader4", 0.0);
F:\ESP8266\v3_1.05\v3_1.05.ino:512:31: warning: deprecated conversion from string constant to 'char*' [-Wwrite-strings]
  sendMessage("/2/fader5", inc);
F:\ESP8266\v3_1.05\v3_1.05.ino:513:31: warning: deprecated conversion from string constant to 'char*' [-Wwrite-strings]
  sendMessage("/2/fader6", inc);
F:\ESP8266\v3_1.05\v3_1.05.ino:514:32: warning: deprecated conversion from string constant to 'char*' [-Wwrite-strings]
  sendMessage("/2/toggle1", 0.0);
F:\ESP8266\v3_1.05\v3_1.05.ino:515:32: warning: deprecated conversion from string constant to 'char*' [-Wwrite-strings]
  sendMessage("/2/toggle2", 0.0);
F:\ESP8266\v3_1.05\v3_1.05.ino: In function 'void PageOne()':
v3_1.05:594: error: 'actionSetup' was not declared in this scope
  actionSetup();
v3_1.05:637: error: 'actionSetup' was not declared in this scope
  actionSetup();
v3_1.05:643: error: 'actionSetup' was not declared in this scope
  actionSetup();
v3_1.05:657: error: 'Kill' was not declared in this scope
  Kill();
exit status 1
'resetControls' was not declared in this scope

[sasa999]

SOLVED!

[matthijskooijman]

Can you tell us how you solved it?

Also, do these wrong line numberings when compiling for the ESP boards? Or do they also occur when you select e.g. the Arduino Uno? I don't think there is anything ESP-specific in the small example you posted? It might be better to test with the Uno, since that prevents any (unrelated) breakage related to the ESP core and newer versions for cluttering the debugging.

[sasa999]

I removed ALL Arduino files (not libraries in My documents/Arduino) from comuter + 14.dec I download Hourly build + esp8266 2.0.0. + deleted some my ESP8266 libraries (can't remember names, but they was shown as inavalid + deleted STM32 from hardware it was shown as invalid too, but worked fine with 1.6.4 + changed some variables, for example it didn't like int8_t reverse (without comments just show error compiation) - i changed to int8_t reverss + something was wrong with my uint32_t time (multiple definition of `time' ) and I changed it to time2 + moved ALL functions up , before loop(). + removed libc_replacements.c with my changes (in 1.6.4 it did not calculate right atan2 and I make some changes for fix it. In latest hourly buils atan2 is calculated right) .. etc. .. cant remember all changes :(
What exactly make wrong line numberings etc. - cant remember at 100% , but I think it was before I moved up my functions from bottom af page , variable names changing.
But very strange that with #define MODE 0 it compiles without problem (v.1.6.5), but for MODE 1 I need 1001 changes..
And question - HourlyBuild from 14.dec is the same 1.6.6 + some fixed bugs??

[matthijskooijman]

And question - HourlyBuild from 14.dec is the same 1.6.6 + some fixed bugs??

Yes, possibly new features too. The hourly build is made from the git master revision, you can see the changes in there using the "commits" tab here in github.

I finally had a bit of time to try your code for myself. I ran it in 1.6.5 (pasting the snippet from your first post above the bare minimum default sketch) and compiled for the Uno, and that broke. The errors were different, but investigating the modifed sketch source (which is actually fed to the compiler), I saw that the function declarations, and the associated #line directive, were inserted at the wrong place, between the #if and #endif. Since then the #line directive would be removed during actual compilation, line numbers would be wrong. Furthermore, the declarations would be removed, causing other errors.

I also tried using the latest nightly, and that compiles fine. The declarations now get inserted at the right place, because their generation was significantly improved in recent versions. Since we now know what the problem was, and it is working for you, I'll close this issue. Thanks for your feedback!