c't

c't-Projekte - Mailinglisten


[Voriger (Datum)] [Nächster (Datum)] [Voriger (Thread)] [Nächster (Thread)]
[Nach Datum][Nach Thread]

Re: [ct-bot] Patch für fehlende #ifdef-Verzweigungen

Absender: Simon
Datum: Mi, 09.03.2011 15:40:49
In-reply-to: <50C3D3CC-FE08-41D2-AB8A-C6B5D07F1A35@xxxxxxxxxxxxxxx>
References: <1299083653.1728.40.camel@wbam-desktop> <50C3D3CC-FE08-41D2-AB8A-C6B5D07F1A35@xxxxxxxxxxxxxxx>


Hallo Timo,

super, danke! Es scheint jetzt alles zu funktionieren. Wir testen es in
den nÃchsten Tagen abschlieÃend. Ich werde Patches in Zukunft direkt an
dich als Anhang schicken. Dann klappt es leichter.

Noch zu Benjamin Benz' Frage: Wir geben einen Einsteigerkurs zur
C-Programmierung im ersten Semester des Studiengangs Elektrotechnik und
Informationstechnik. Die Roboterprogrammierung ist da ein Zusatzangebot.
Es soll zum einen einzelne Studenten motivieren, etwas abseits des
Ãblichen AnfÃngerpfads zu schauen, und zum anderen Teilnehmer
ansprechen, die schon Ãber Vorkenntnisse verfÃgen.

Den c't-Code gibt es ja fÃr den Simulator und die Hardware. Wir basteln
fÃr den Kurs im Moment an einer dritten Variante fÃr die Kommunikation
Ãber STDIN/STDOUT. Damit wollen wir Code automatisiert testen kÃnnen
(eine Art Mock-Bibliothek, die die Hardware vereinfacht ersetzt).

GruÃ, Simon


Am Mittwoch, den 09.03.2011, 14:19 +0100 schrieb Timo Sandmann:
> Hallo Simon,
> 
> vielen Dank fÃr den Patch und den Hinweis auf den Bug. 
> 
> Der Patch ist jetzt auch ins SVN eingeflossen, ich habe nur noch zwei ganz kleine Ãnderungen mit eingebaut:
> 1. wird cmd_init() nicht komplett mit #ifdef BEHAVIOUR_REMOTECALL_AVAILABLE deaktiviert, das erspart die Unterscheidung in ctbot_init_low_last() und fÃhrt nicht dazu, dass es zwar immer eine Deklaration von cmd_init() gibt, aber nur manchmal auch eine Definition der Funktion.
> 2. ist in gui_display() auch LED_off(LED_WEISS); von RC5_AVAILABLE abhÃngig. Wenn RC5 aus ist, mÃchte man die weiÃe LED ja eventuell anderweitig benutzen. 
> 
> 
> Noch eine organisatorische Bitte: Da Patches vom Mailsystem umformatiert werden (zusÃtzliche ZeilenumbrÃche, fÃhrende Leerzeichen fehlen) und dann nicht mehr direkt funktionieren, den Patch am besten in eine Zip-Datei verpacken. 
> Allerdings bin ich mir nicht sicher, ob die Mailingliste zip-AnhÃnge mit ausliefert? Das sollte vielleicht mal ÃberprÃft und angepasst werden. Zur Not den Anhang unter http://www.heise.de/ct/projekte/machmit/ctbot/wiki/MLAttachments hochladen oder einfach direkt verschicken. Sorry fÃr die UmstÃnde.
> 
> 
> SchÃne GrÃÃe,
> Timo
> 
> 
> Am 02.03.2011 um 17:34 schrieb Simon Siemens:
> > Hallo Mailing-Liste,
> > 
> > ich habe den ct-Bot-Code des stable-Zweigs mit einer sehr eingeschrÃnkten Konfiguration in ct-Bot.h Ãbersetzt. Dabei sind einige Linker-Fehler aufgetreten, die ich alle auf fehlende #ifdef-Verzweigungen zurÃckfÃhren konnte. Am Ende dieser Email finden Sie einen Patch, der diese Fehler korrigiert. 
> > 
> > Kann bitte jemand den Patch prÃfen? Insbesondere durch die Bedingungen fÃr
> > BEHAVIOUR_REMOTECALL_AVAILABLE fÃllt in cmd-tools_pc.c einiges heraus.
> > Mir scheint das so korrekt zu sein.
> > 
> > Ich wÃrde mich sehr freuen, wenn die Fehler noch im aktuellen
> > stable-Zweig korrigiert werden wÃrden. Damit ist es fÃr mich und meine
> > Studenten leichter, eine funktionierende Codebasis herzustellen.
> > 
> > GruÃ, Simon
> 
> 
> 
> _______________________________________________
> ct-bot-entwickler Mailingliste
> ct-bot-entwickler@xxxxxxxxxxxxxxxxx
> http://www.heise.de/bin/newsletter/listinfo/ct-bot-entwickler