# maria-developers team mailing list archive

## Removing a bunch of windows compilation warnings

```Hi everyone,

```
I have been working on getting rid of the compiler warnings on Windows. I'm not sure if I have the proper fix for this one, so I separated it from the others.
```
TAILQ_EMPTY(head) is defined in extra\libevent\event-internal.h as this:

It's also defined in extra\libevent\compat\sys\queue.h as:

```
As you can see, these two are actually identical, but the Visual C++ compiler still complains about them, because it doesn't understand that they are identical.
```
```
There are several ways to fix it. The proper way would be to avoid having the same define in two different places, but I don't know why it's there in the first place. The second is to guard both defines with an #ifndef TAILQ_EMPTY, which works fine, of course. The one I've chosen is to change the definition in queue.h to be identical (character wise) to the other, and this shuts up the compiler.
```
This is the patch:

=== modified file 'extra/libevent/compat/sys/queue.h'
--- extra/libevent/compat/sys/queue.h   2009-03-12 22:27:35 +0000
+++ extra/libevent/compat/sys/queue.h   2010-04-09 09:34:31 +0000
@@ -298,9 +298,7 @@
/* XXX */
```
```-       (TAILQ_FIRST(head) == TAILQ_END(head))
-
```
```        for((var) = TAILQ_FIRST(head);                                  \

Ok to push to lp:maria?

Bo.

```