From 528dd4b81f794912f96c83bc7b07d55c842f765d Mon Sep 17 00:00:00 2001 From: Enno Rehling Date: Thu, 6 Mar 2014 19:12:12 -0800 Subject: [PATCH] remove latin1 characters from some files experimenting with .gitattributes some more --- .gitattributes | 1 + src/spy.c | 20 ++++++++--------- src/study.c | 58 +++++++++++++++++++++++++------------------------- src/summary.c | 2 +- todo.md | 19 +++++++++++++++++ 5 files changed, 60 insertions(+), 40 deletions(-) create mode 100644 todo.md diff --git a/.gitattributes b/.gitattributes index a827124ce..a62b543f8 100644 --- a/.gitattributes +++ b/.gitattributes @@ -1,3 +1,4 @@ +*.[hc] text *.json text *.xml text *.py text diff --git a/src/spy.c b/src/spy.c index fa5116060..2a277484a 100644 --- a/src/spy.c +++ b/src/spy.c @@ -65,7 +65,7 @@ void spy_message(int spy, const unit * u, const unit * target) target, str)); if (spy > 20) { sc_mage *mage = get_mage(target); - /* bei Magiern Zaubersprüche und Magiegebiet */ + /* for mages, spells and magic school */ if (mage) { ADDMSG(&u->faction->msgs, msg_message("spyreport_mage", "target type", target, magic_school[mage->magietyp])); @@ -74,7 +74,7 @@ void spy_message(int spy, const unit * u, const unit * target) if (spy > 6) { faction *fv = visible_faction(u->faction, target); if (fv && fv != target->faction) { - /* wahre Partei */ + /* true faction */ ADDMSG(&u->faction->msgs, msg_message("spyreport_faction", "target faction", target, target->faction)); ql_set_insert(&u->faction->seen_factions, target->faction); @@ -138,9 +138,9 @@ int spy_cmd(unit * u, struct order *ord) cmistake(u, u->thisorder, 39, MSG_EVENT); return 0; } - /* Die Grundchance für einen erfolgreichen Spionage-Versuch ist 10%. - * Für jeden Talentpunkt, den das Spionagetalent das Tarnungstalent - * des Opfers übersteigt, erhöht sich dieses um 5%*/ + /* Die Grundchance fuer einen erfolgreichen Spionage-Versuch ist 10%. + * Fuer jeden Talentpunkt, den das Spionagetalent das Tarnungstalent + * des Opfers uebersteigt, erhoeht sich dieses um 5%*/ spy = eff_skill(u, SK_SPY, r) - eff_skill(target, SK_STEALTH, r); spychance = 0.1 + MAX(spy * 0.05, 0.0); @@ -160,8 +160,8 @@ int spy_cmd(unit * u, struct order *ord) observe = MIN(observe, 0); } - /* Anschließend wird - unabhängig vom Erfolg - gewürfelt, ob der - * Spionageversuch bemerkt wurde. Die Wahrscheinlich dafür ist (100 - + /* Anschliessend wird - unabhaengig vom Erfolg - gewuerfelt, ob der + * Spionageversuch bemerkt wurde. Die Wahrscheinlich dafuer ist (100 - * SpionageSpion*5 + WahrnehmungOpfer*2)%. */ observechance = 1.0 - (eff_skill(u, SK_SPY, r) * 0.05) + (eff_skill(target, SK_PERCEPTION, r) * 0.02); @@ -238,7 +238,7 @@ int setstealth_cmd(unit * u, struct order *ord) trace = findrace(s, u->faction->locale); if (trace) { - /* Dämonen können sich nur als andere Spielerrassen tarnen */ + /* demons can cloak as other player-races */ if (u_race(u) == new_race[RC_DAEMON]) { race_t allowed[] = { RC_DWARF, RC_ELF, RC_ORC, RC_GOBLIN, RC_HUMAN, RC_TROLL, RC_DAEMON, RC_INSECT, RC_HALFLING, RC_CAT, RC_AQUARIAN, @@ -256,7 +256,7 @@ int setstealth_cmd(unit * u, struct order *ord) return 0; } - /* Singdrachen können sich nur als Drachen tarnen */ + /* Singdrachen koennen sich nur als Drachen tarnen */ if (u_race(u) == new_race[RC_SONGDRAGON] || u_race(u) == new_race[RC_BIRTHDAYDRAGON]) { if (trace == new_race[RC_SONGDRAGON] || trace == new_race[RC_FIREDRAGON] @@ -268,7 +268,7 @@ int setstealth_cmd(unit * u, struct order *ord) return 0; } - /* Dämomen und Illusionsparteien können sich als andere race tarnen */ + /* Daemomen und Illusionsparteien koennen sich als andere race tarnen */ if (u_race(u)->flags & RCF_SHAPESHIFT) { if (playerrace(trace)) { u->irace = trace; diff --git a/src/study.c b/src/study.c index 4c20dd090..fffde25bf 100644 --- a/src/study.c +++ b/src/study.c @@ -86,7 +86,7 @@ magic_t getmagicskill(const struct locale * lang) } /* ------------------------------------------------------------- */ -/* Vertraute und Kröten sind keine Migranten */ +/* familiars and toads are not migrants */ bool is_migrant(unit * u) { if (u_race(u) == u->faction->race) @@ -131,7 +131,7 @@ int study_cost(unit * u, skill_t sk) case SK_ALCHEMY: return 200; case SK_MAGIC: /* Die Magiekosten betragen 50+Summe(50*Stufe) */ - /* 'Stufe' ist dabei die nächste zu erreichende Stufe */ + /* 'Stufe' ist dabei die naechste zu erreichende Stufe */ stufe = 1 + get_level(u, SK_MAGIC); return k * (1 + ((stufe + 1) * stufe / 2)); default: @@ -181,10 +181,10 @@ teach_unit(unit * teacher, unit * student, int nteaching, skill_t sk, int n; /* learning sind die Tage, die sie schon durch andere Lehrer zugute - * geschrieben bekommen haben. Total darf dies nicht über 30 Tage pro Mann + * geschrieben bekommen haben. Total darf dies nicht ueber 30 Tage pro Mann * steigen. * - * n ist die Anzahl zusätzlich gelernter Tage. n darf max. die Differenz + * n ist die Anzahl zusaetzlich gelernter Tage. n darf max. die Differenz * von schon gelernten Tagen zum MAX(30 Tage pro Mann) betragen. */ if (magic_lowskill(student)) { @@ -219,36 +219,36 @@ teach_unit(unit * teacher, unit * student, int nteaching, skill_t sk, teach->teachers[index] = NULL; teach->value += n; - /* Solange Akademien größenbeschränkt sind, sollte Lehrer und - * Student auch in unterschiedlichen Gebäuden stehen dürfen */ + /* Solange Akademien groessenbeschraenkt sind, sollte Lehrer und + * Student auch in unterschiedlichen Gebaeuden stehen duerfen */ if (btype == bt_find("academy") && student->building && student->building->type == bt_find("academy")) { int j = study_cost(student, sk); j = MAX(50, j * 2); /* kann Einheit das zahlen? */ if (get_pooled(student, oldresourcetype[R_SILVER], GET_DEFAULT, j) >= j) { - /* Jeder Schüler zusätzlich +10 Tage wenn in Uni. */ - teach->value += (n / 30) * 10; /* learning erhöhen */ - /* Lehrer zusätzlich +1 Tag pro Schüler. */ + /* Jeder Schueler zusaetzlich +10 Tage wenn in Uni. */ + teach->value += (n / 30) * 10; /* learning erhoehen */ + /* Lehrer zusaetzlich +1 Tag pro Schueler. */ if (academy) *academy += n; } /* sonst nehmen sie nicht am Unterricht teil */ } /* Teaching ist die Anzahl Leute, denen man noch was beibringen kann. Da - * hier nicht n verwendet wird, werden die Leute gezählt und nicht die + * hier nicht n verwendet wird, werden die Leute gezaehlt und nicht die * effektiv gelernten Tage. -> FALSCH ? (ENNO) * * Eine Einheit A von 11 Mann mit Talent 0 profitiert vom ersten Lehrer B - * also 10x30=300 tage, und der zweite Lehrer C lehrt für nur noch 1x30=30 - * Tage (damit das Maximum von 11x30=330 nicht überschritten wird). + * also 10x30=300 tage, und der zweite Lehrer C lehrt fuer nur noch 1x30=30 + * Tage (damit das Maximum von 11x30=330 nicht ueberschritten wird). * - * Damit es aber in der Ausführung nicht auf die Reihenfolge drauf ankommt, + * Damit es aber in der Ausfuehrung nicht auf die Reihenfolge drauf ankommt, * darf der zweite Lehrer C keine weiteren Einheiten D mehr lehren. Also * wird student 30 Tage gutgeschrieben, aber teaching sinkt auf 0 (300-11x30 <= * 0). * - * Sonst träte dies auf: + * Sonst traete dies auf: * * A: lernt B: lehrt A C: lehrt A D D: lernt * @@ -258,11 +258,11 @@ teach_unit(unit * teacher, unit * student, int nteaching, skill_t sk, * Ist C aber vor B dran, lehrt C 300 tage an A, und 0 tage an D, * und B lehrt auch 0 tage an A. * - * Deswegen darf C D nie lehren dürfen. + * Deswegen darf C D nie lehren duerfen. * * -> Das ist wirr. wer hat das entworfen? - * Besser wäre, man macht erst vorab alle zuordnungen, und dann - * die Talentänderung (enno). + * Besser waere, man macht erst vorab alle zuordnungen, und dann + * die Talentaenderung (enno). */ nteaching = MAX(0, nteaching - student->number * 30); @@ -303,7 +303,7 @@ int teach_cmd(unit * u, struct order *ord) if ((i = get_effect(u, oldpotiontype[P_FOOL])) > 0) { /* Trank "Dumpfbackenbrot" */ i = MIN(i, u->number * TEACHNUMBER); - /* Trank wirkt pro Schüler, nicht pro Lehrer */ + /* Trank wirkt pro Schueler, nicht pro Lehrer */ teaching -= i * 30; change_effect(u, oldpotiontype[P_FOOL], -i); j = teaching / 30; @@ -434,7 +434,7 @@ int teach_cmd(unit * u, struct order *ord) || alliedunit(u2, u->faction, HELP_GUARD); /* Neuen Befehl zusammenbauen. TEMP-Einheiten werden automatisch in - * ihre neuen Nummern übersetzt. */ + * ihre neuen Nummern uebersetzt. */ if (zOrder[0]) strcat(zOrder, " "); strcat(zOrder, unitid(u2)); @@ -560,7 +560,7 @@ int learn_cmd(unit * u, order * ord) cmistake(u, ord, 771, MSG_EVENT); return 0; } - /* Hack: Talente mit Malus -99 können nicht gelernt werden */ + /* Hack: Talente mit Malus -99 koennen nicht gelernt werden */ if (u_race(u)->bonus[sk] == -99) { cmistake(u, ord, 771, MSG_EVENT); return 0; @@ -574,7 +574,7 @@ int learn_cmd(unit * u, order * ord) } } - /* snotlings können Talente nur bis T8 lernen */ + /* snotlings koennen Talente nur bis T8 lernen */ if (u_race(u) == new_race[RC_SNOTLING]) { if (get_level(u, sk) >= 8) { cmistake(u, ord, 308, MSG_EVENT); @@ -588,9 +588,9 @@ int learn_cmd(unit * u, order * ord) teach = (teaching_info *) a->data.v; } - /* keine kostenpflichtigen Talente für Migranten. Vertraute sind + /* keine kostenpflichtigen Talente fuer Migranten. Vertraute sind * keine Migranten, wird in is_migrant abgefangen. Vorsicht, - * studycost darf hier noch nicht durch Akademie erhöht sein */ + * studycost darf hier noch nicht durch Akademie erhoeht sein */ if (studycost > 0 && !ExpensiveMigrants() && is_migrant(u)) { ADDMSG(&u->faction->msgs, msg_feedback(u, ord, "error_migrants_nolearn", "")); @@ -612,8 +612,8 @@ int learn_cmd(unit * u, order * ord) return 0; } if (is_familiar(u)) { - /* Vertraute zählen nicht zu den Magiern einer Partei, - * können aber nur Graue Magie lernen */ + /* Vertraute zaehlen nicht zu den Magiern einer Partei, + * koennen aber nur Graue Magie lernen */ mtyp = M_GRAY; if (!is_mage(u)) create_mage(u, mtyp); @@ -628,12 +628,12 @@ int learn_cmd(unit * u, order * ord) mtyp = getmagicskill(u->faction->locale); if (mtyp == M_NONE || mtyp == M_GRAY) { /* wurde kein Magiegebiet angegeben, wird davon - * ausgegangen, daß das normal gelernt werden soll */ + * ausgegangen, dass das normal gelernt werden soll */ if (u->faction->magiegebiet != 0) { mtyp = u->faction->magiegebiet; } else { /* Es wurde kein Magiegebiet angegeben und die Partei - * hat noch keins gewählt. */ + * hat noch keins gewaehlt. */ mtyp = getmagicskill(u->faction->locale); if (mtyp == M_NONE) { cmistake(u, ord, 178, MSG_MAGIC); @@ -658,7 +658,7 @@ int learn_cmd(unit * u, order * ord) } else { /* ist schon ein Magier und kein Vertrauter */ if (u->faction->magiegebiet == 0) { - /* die Partei hat noch kein Magiegebiet gewählt. */ + /* die Partei hat noch kein Magiegebiet gewaehlt. */ mtyp = getmagicskill(u->faction->locale); if (mtyp == M_NONE) { mtyp = getmagicskill(u->faction->locale); @@ -779,7 +779,7 @@ int learn_cmd(unit * u, order * ord) } fset(u, UFL_LONGACTION | UFL_NOTMOVING); - /* Anzeigen neuer Tränke */ + /* Anzeigen neuer Traenke */ /* Spruchlistenaktualiesierung ist in Regeneration */ if (sk == SK_ALCHEMY) { diff --git a/src/summary.c b/src/summary.c index 8ac9d546a..eedab42e2 100644 --- a/src/summary.c +++ b/src/summary.c @@ -339,7 +339,7 @@ summary *make_summary(void) } } - /* Alles zählen */ + /* count everything */ for (r = regions; r; r = r->next) { s->pferde += rhorses(r); diff --git a/todo.md b/todo.md new file mode 100644 index 000000000..ec14a6843 --- /dev/null +++ b/todo.md @@ -0,0 +1,19 @@ +## Eliminate non-ascii characters from the code + +In Linux, file *.c will output this: + + study.c: C source, ISO-8859 text + summary.c: C source, ISO-8859 text + test_eressea.c: C source, ASCII text + +Let's make all the source files ASCII. In a lot of cases, that means +rewriting German comments to be English, which is also great. + +## remove platform.h, use rtl.h + +In Atlantis, I use a header called rtl.h to polyfill functions like +_strdup and _snprintf. I would like that to be a reusable library that +Eressea can use, and eliminate the hacks in platform.h in favor of +autoconf-style detections done by CMake (already happening for a few +things). +