Druckbare Version
Ich habe mir die neueste WIN AVR installiert, und beim 1. compillieren erschein dieser WarnhinweisIch hab mir zwar auch die LIB 2.72. runtergeladen, aber die sagt mir WIN AVR nicht vorhanden. Alles etwas verwirrend.Code:from test.c:1:
c:/winavr-20090313/lib/gcc/../../avr/include/avr/signal.h:36:2: warning: #warning "This header file is obsolete. Use <avr/interrupt.h>."
Hat jemand ne Idee?
Beim 2.compillieren erscheint dan die Meldung nicht mehr
Das ist ein Hinweis auf einen Änderung des GCC in Bezug auf die ISR-Aufrufe. Siehe:
file:///C:/WinAVR-20090313/doc/avr-libc/avr-libc-user-manual/group__avr__interrupts.html
Leider hast du ein link gemacht zu einer datei auf deine festplatte. Nur die das auch dort gespeichert haben werden es finden. Das pfad in dem link ist:Zitat:
Zitat von Hubert.G
C:/WinAVR-20090313/doc/avr-libc/avr-libc-user-manual/group__avr__interrupts.html
Der Link sollte wenigstens bei fank1210 passen, ist ja teil des WINAVR Paketes.
Das Problem ist eine Änderung wie Interrupts eingebunden werden. Das Programm ist wohl noch für eine ältere Compilerversion (Gcc 3.xx). Man sollte also eventuell das Programm anpassen.
Wenn man zweimal kompiliert, verschwinden die Warnings immer alle. Das ist ein Feature, damit man bei vielen Warnings die Fehler filtern kann.
Irgendwie hatte ich mich bisher immer wieder gefragt, warum das so sei. Danke!Zitat:
Zitat von thewulf00
hm... Aber ist das nicht irgendwie der "falsche" Ansatz, dass man erst die Fehler beseitigt, dann die Warnungen?
Sicher kann man das diskutieren. Aber ich weiß, dass IMMER beim 2. Kompilieren die Warnings verschwinden - und zwar nicht nur durch die Probleme, die der OP hat.
OK, und was kann ich jetzt wo einstellen, oder ist der Warnhinweis zu ignorieren?
Nicht ignorieren. Einfach nicht mehr die signal.h einbinden, sondern die interrupt.h.
Ja und wie, im asuro.h habe ich folgendes gefunden:
#include <avr/io.h>
#include <avr/interrupt.h>
#include <avr/signal.h>
Dann ist die Software für den Asuro mit einem älteren GCC getestet worden. Also entweder machst Du einen Downgrade, oder Du kommentierst die Zeile aus:
#include <avr/io.h>
#include <avr/interrupt.h>
//#include <avr/signal.h>
Das ist wohl die sauberste Lösung.Zitat:
#include <avr/interrupt.h>
//#include <avr/signal.h>
Das wußte ich auch nicht. Aber das liegt wohl auch daran, dass ich teilweise komplett umgebaute Programme beim ersten Versuch schon fast fehler- und warnungsfrei übersetzen kann.Zitat:
... ein Feature, damit man bei vielen Warnings die Fehler filtern kann.
Gruß
mic
Meinen Respekt, radbruch!
Ich kann das nichtmal bei meinen eigenen... :)
Und noch was,die Funktion speed müsste doch im ASURO.H definiert sein.
Warum kommt dan dieser Warnhinweis:
test.c:102: warning: operation on 'speed' may be undefined
test.c:103: warning: operation on 'speed' may be undefined
Las deine code bitte sehen. Ohne das mussen wir auf unsere glasskugel verzichten. Und der fehlt manchmal. ;)
Im asuro.h oder eben asuro.c gibts gar keinen variabele oder funktion 'speed'. Nur die funktion MotorSpeed.
Hier die Stelle mit den Fehlern. Ist außerdem aus dem Buch "SPASS MIT ASURO BAND ":
Code:if (difference<-speed)difference=-speed;
if (difference>speed)difference=speed;
if (difference>0) MotorSpeed(speed,speed++);
else MotorSpeed(speed,speed--);
}
return 0;
}
Wenn die IFs nicht zutreffen hat speed keinen definierten Wert. Etwas wie
unsigned char speed=100;
fehlt hier.
ist denke ich mal doch alles vorhanden:
Code:#include "asuro.h"
#define SCHWELLEL 582
#define SCHWELLER 549
#define ANPASSUNG 10
#define LOW 0
#define HIGH 1
int main (void){
unsigned int data[2];
unsigned long int count=0;
signed int status[2]={0,0};
signed int difference=0;
signed int speed=200;
int i=0;
Init();
while(1){
switch(count)
{
case 0: MotorDir(FWD,FWD);speed=200;break;
case 249: MotorDir(BREAK,BREAK);
for(i=0;i<282;i++){Sleep(255);}
MotorDir(FWD,BREAK);speed=200;count=250;break;
case 330: MotorDir(BREAK,BREAK);
}
OdometrieData(data);
if ((status[0]==LOW)&&(data[0]>SCHWELLEL+ANPASSUNG))
{
status[0]=HIGH;
difference++;
count++;
}
if ((status[0]==HIGH)&&(data[0]<SCHWELLEL-ANPASSUNG))
{
status[0]=LOW;
difference++;
count++;
}
if ((status[1]==LOW)&&(data[1]>SCHWELLER+ANPASSUNG))
{
status[1]=HIGH;
difference--;
}
if ((status[1]==HIGH)&&(data[1]<SCHWELLER-ANPASSUNG))
{
status[1]=LOW;
difference--;
}
if (difference<-speed)difference=-speed;
if (difference>speed)difference=speed;
if (difference>0) MotorSpeed(speed,speed++);
else MotorSpeed(speed,speed--);
}
return 0;
}
Ist das der orginale Code aus dem Buch? Die Bildung von difference aus den Odometriedaten scheint mir ja noch plausibel, aber das hier ist doch fragwürdig:
Wenn der Odozählerunterschied kleiner -speed dann Unterschied gleich -speedCode:if (difference<-speed)difference=-speed;
if (difference>speed)difference=speed;
if (difference>0) MotorSpeed(speed,speed++);
else MotorSpeed(speed,speed--);
Wenn der Odozählerunterschied größer speed dann Unterschied gleich speed
Das "undefiniert" bezieht sich übrigens auf das ++ (bzw. --) hier:
MotorSpeed(speed,speed++);
Warum das der Kompiler anmotzt ist mir auch unklar.
Gruß
mic
so sieht das Original aus:
mit dem (++)bzw.(--)habe ich rumexperimentiert.Code:if (difference<-speed)difference=-speed;
if (difference>speed)difference=speed;
if (difference>0) MotorSpeed(speed-difference,speed);
else MotorSpeed(speed,speed+difference);
Gruß Frank
Stimmt es du arbeitest mit den code auf seite 18, kapitel 1.5.2 "Nur ein Stükchen geradeaus"?
Hmm, MotorSpeed erwartet "unsigned char", speed ist aber ein "signed int". Ein "--" operation auf speed wan es die wert 0 hat wird ein 'undefinierte' auswirkung geben. Deshalb die warnung denke ich.
Ein warnung ist auch nur ein warnung. Ein run-time fehler (fehler im betrieb) wird nicht unbedingt auftretten. De compiler macht dich aufmerksam auf ein potentieles problem das du besser mit weitere kontrollier-code vermeiden soll.