Hi Udo,
As matter of style, I see you add extra spaces around if expressions like:
- if (bt1 == VT_LLONG)
+ if ( bt1 == VT_LLONG )
type.t &= t1;
- if (bt2 == VT_LLONG)
+ if ( bt2 == VT_LLONG )
Which is a very uncommon programming style in tcc code
~/tinycc $ fgrep 'if ( ' *.c *.h
tccelf.c: if ( ref->handle )
tccgen.c: else if ( (s->type.t & VT_BTYPE) == VT_FUNC ) {
tccgen.c: if ( ad.cleanup_func ) {
tccgen.c: if ( (vtop->type.t & VT_BTYPE) != VT_STRUCT ) {
tccgen.c: } else if ( bt1 == VT_BOOL && bt2 == VT_BOOL ) {
tccgen.c: } else if ( is_float(bt1) && is_integer_btype(bt2) ) {
tccgen.c: } else if ( is_integer_btype(bt1) && is_float(bt2) ) {
tccgen.c: if ( bt1 == VT_LLONG )
tccgen.c: if ( bt2 == VT_LLONG )
tccgen.c: else if ( bt1 == VT_PTR && is_null_pointer(vtop) ) {
tccgen.c: } else if ( is_null_pointer(&sv) && bt2 == VT_PTR ) {
tccgen.c: } else if ( bt1 == VT_PTR && is_integer_btype(bt2) ) {
tccgen.c: } else if ( is_integer_btype(bt1) && bt2 == VT_PTR ) {
tccgen.c: } else if ( bt1==VT_PTR && bt2 == VT_PTR ) {
tccgen.c: if ( type1.ref != type2.ref ) {
tccgen.c: } else if ( is_integer_btype(bt1) && is_integer_btype(bt2)
) {
tccpe.c: if ( !dllref->handle )
C.
From: Tinycc-devel [mailto:tinycc-devel-bounces+eligis=orange...@nongnu.org] On
Behalf Of Udo Pfrengle
Sent: Monday, February 17, 2020 23:35
To: tinycc-devel@nongnu.org
Subject: *** SPAM *** [Tinycc-devel] My changes
Hi folks,
I hope I do not too much nonsense with my changes.
No clue why there appeared a merge from me, that was a mistake - I am new to
git and still learning it.
I am really excited about the speed and terseness of tcc and would like to help
to improve it further.
Have you any recommendations? Are there some weak spots which need some work?
... and I unintentionally disclosed my eMail address. Is there a way to remove
it from the git log again?
Kind regards
Udo
_______________________________________________
Tinycc-devel mailing list
Tinycc-devel@nongnu.org
https://lists.nongnu.org/mailman/listinfo/tinycc-devel
