diff options
| author | Chun-wei Fan <fanchunwei@src.gnome.org> | 2021-04-02 12:37:47 +0800 |
|---|---|---|
| committer | Emmanuele Bassi <ebassi@gmail.com> | 2022-02-13 15:19:43 +0000 |
| commit | c0098000f69020732e416caf5309b38d18cf8b34 (patch) | |
| tree | 3ca8e916927294536b8836c3cac07292f02e3564 | |
| parent | 5abb0eb00c7d43486606c83ab962746b4e291378 (diff) | |
| download | gobject-introspection-c0098000f69020732e416caf5309b38d18cf8b34.tar.gz | |
bdz.c: Some cleanups
We can just update the for loop condition to be >0 for all builds, which
is actually equivilant to >=1 as we are essentially comparing an
unsigned 32-bit int, so that we don't need to worry about fixing the
VS2012 bug invasively, as Visual Studio 2012 x64 is more sensitive about
sizes of variables (e.g. pointer sizes in this case)
| -rw-r--r-- | girepository/cmph/bdz.c | 10 |
1 files changed, 1 insertions, 9 deletions
diff --git a/girepository/cmph/bdz.c b/girepository/cmph/bdz.c index 34898ad5..e70f1183 100644 --- a/girepository/cmph/bdz.c +++ b/girepository/cmph/bdz.c @@ -423,7 +423,7 @@ static void assigning(bdz_config_data_t *bdz, bdz_graph3_t* graph3, bdz_queue_t memset(marked_vertices, 0, (size_t)(bdz->n >> 3) + 1); memset(bdz->g, 0xff, (size_t)(sizeg)); - for(i=nedges-1;i+1>=1;i--){ + for(i=nedges-1;i+1>0;i--){ curr_edge=queue[i]; v0=graph3->edges[curr_edge].vertices[0]; v1=graph3->edges[curr_edge].vertices[1]; @@ -455,14 +455,6 @@ static void assigning(bdz_config_data_t *bdz, bdz_graph3_t* graph3, bdz_queue_t SETBIT(marked_vertices, v2); } DEBUGP("A:%u %u %u -- %u %u %u\n", v0, v1, v2, GETVALUE(bdz->g, v0), GETVALUE(bdz->g, v1), GETVALUE(bdz->g, v2)); - -#if (_MSC_VER > 1699 && _MSC_VER < 1800) - /* This is bad, MSVC 2012 X64 getting confused with the value of i... */ - /* an obvious MSVC 2012 X64 compiler bug :| */ - if (i <= 0) - break; -#endif - }; free(marked_vertices); } |