99136a16e7
inp2m.c, #11/15b, rewrite, use array expressions
9 years ago
7eb6006461
inp2m.c, #10/15b, use 'nodeflag' instead of literal values
9 years ago
7ab64775b8
inp2m.c, #9/15b, rewrite, use arrays for 'node' and 'mname'
9 years ago
395939d2f4
inp2m.c, #8/15b, unify, add two invocations without any effect
9 years ago
55c9c3a73b
inp2m.c, #7/15b, cleanup some braces
9 years ago
7a1b65f045
inp2m.c, #6/15b, abstraction, introduce valid_numnodes()
9 years ago
cff00524c2
inp2m.c, #5/15b, express some facts in comments for the following commits
9 years ago
0a95935e46
inp2m.c, #4/15b, unify code slightly
9 years ago
f7b3f40c70
inp2m.c, #3/15b, set 'save' a little sooner
9 years ago
40c632f8a7
inp2m.c, #2/15b, drop TRACE code
9 years ago
f9213e3fe4
inp2m.c, #1/15b, drop comments
9 years ago
6fde9353ea
inp2m.c, #7/7, whitespace
9 years ago
11572fe9d0
inp2m.c, #6/7, polish comments
9 years ago
f66d37463f
inp2m.c, #5/7, renumber `nodeflag'
9 years ago
6a275a6bd9
inp2m.c, #4/7, rewrite
9 years ago
08b798c485
inp2m.c, #3/7, line wraps
9 years ago
f23876e30c
inp2m.c, #2/7, cleanup braces
9 years ago
80f58e98a8
inp2m.c, #1/7, cleanup comparisons
9 years ago
a2347a28d6
inp2*.c, cleanup, drop redundant initialisation
which can be dropped to the recently fixed INPgetMod(), see
inpgmod.c, INPgetMod(), bug fix, missing 'model' assignment
Note:
there have been INPgetMod() invocations without initialisation,
causing access to non-initialised memory with the unfixed INPgetMod()
9 years ago
56b2342df3
inpgmod.c, #15/15, whitespace
9 years ago
fccdea429c
inpgmod.c, #14/15, cleanup
9 years ago
0245f304ff
inpgmod.c, #13/15, shortcut #if CIDER
9 years ago
a8ffd52c42
inpgmod.c, #12/15, cleanup braces
9 years ago
627ed98464
inpgmod.c, #11/15, reduce scope and rewrite
9 years ago
66976d492e
inpgmod.c, #10/15, reorder 'extern's
9 years ago
19e36dd4cb
inpgmod.c, #9/15, drop trailing whitespace in messages
9 years ago
370420d00e
inpgmod.c, #8/15, commute boolean expressions
9 years ago
4842cd39a7
inpgmod.c, #7/15, reduce scope and rewrite
9 years ago
3212b18115
inpgmod.c, #6/15, simplify return of boolean expressions
9 years ago
6ee483ed1d
inpgmod.c, #5/15, simplify tprintf usage
9 years ago
b329fd56fc
inpgmod.c, #4/15, unify 'return' statements
9 years ago
cdc8fbd192
inpgmod.c, #3/15, line wraps
9 years ago
b0537a7e96
inpgmod.c, #2/15, polish comments
9 years ago
6cdd306fea
inpgmod.c, #1/15, cleanup comparisons
9 years ago
5905ed26fa
inpgmod.c, INPgetMod(), bug fix, missing 'model' assignment
'model' is a return value, and must always be assigned to.
NULL denotes the error case.
9 years ago
8b98b9b350
use 'fabs()'
10 years ago
51fa006c63
INPparseNumMod(), #14/14, whitespace cleanup
10 years ago
0b1c79172c
INPparseNumMod(), #13/14, invert 'if' statement
10 years ago
aa5afccf15
INPparseNumMod(), #12/14, use 'info' instead of 'lastType'
to signify having read a cardName
10 years ago
c4d39d66de
INPparseNumMod(), #11/14, remove unused 'cardType'
10 years ago
521fd16c11
INPparseNumMod(), #10/14, don't fall through into the 'default:'
instead, duplicate the code of the 'default:' and break
10 years ago
6b305744d9
INPparseNumMod(), #9/14, invert the 'if' statement
10 years ago
1483e2df9a
INPparseNumMod(), #8/14, move the 'default:' body out of the 'switch'
this body is inevitable, as there is no other break out of the 'switch'
10 years ago
1a9bcf6b33
INPparseNumMod(), #7/14, use 'lastType' instead of the identical 'cardType'
there is no further access to cardType
thus can drop the reduntandant assignments to cardType
10 years ago
110f16b361
INPparseNumMod(), #6/14, again, assign 'lastType' sooner
in the block in-between those two positions
there is no access to lastType
there is no 'continue'
there is no 'break' out of the enclosing switch
thus the assignment is inevitable
there is one position where cardType is modified,
need to assign to lastType there too to keep it in sync
10 years ago
18d6679c02
INPparseNumMod(), #5/14, use 'lastType' instead of the identical 'cardType'
10 years ago
11f3907139
INPparseNumMod(), #4/14, assign 'lastType' sooner
in the block in-between those two positions
there is no access to lastType or cardType
there is no 'continue'
there is no 'break' out of the enclosing 'switch'
thus the assignment is inevitable
10 years ago
7862a4f59f
INPparseNumMod(), #3/14, `continue' the loop instead of
'break'ing out of the `switch'
this is equivalent because the 'switch' is the last statement in the loop body
10 years ago
c1c7b5305e
INPparseNumMod(), #2/14, transform the `while' loop into a `for' loop
there was no 'continue' for this loop
there was no 'break' out of this loop
there was no assignment to txtCard, thus txtCard can't be NULL
10 years ago
e824cbd193
INPparseNumMod(), #1/14, return sooner
setting txtCard to 0 and cardType to E_MISSING (which is < 0)
caused the following `if (cardType >= 0)' to be skipped
and then broke out of the enclosing 'while' loop
in this case, there where no other statements executed
10 years ago
rlar