Re: [Wireshark-dev] [Wireshark-commits] rev 52701: /trunk/epan/ /trunk/epan/: oids_test.c

2013-10-21 Thread Joerg Mayer
On Sun, Oct 20, 2013 at 02:18:19AM +, eapa...@wireshark.org wrote:
 http://anonsvn.wireshark.org/viewvc/viewvc.cgi?view=revrevision=52701
 
 User: eapache
 Date: 2013/10/20 02:18 AM
 
 Log:
  Don't use g_assert_cmpint, it isn't happy on Windows. g_assert is nearly as 
 good
  except it doesn't produce as nice error messages.

How about adding it to check api?

 Ciao
   Jörg
-- 
Joerg Mayer   jma...@loplof.de
We are stuck with technology when what we really want is just stuff that
works. Some say that should read Microsoft instead of technology.
___
Sent via:Wireshark-dev mailing list wireshark-dev@wireshark.org
Archives:http://www.wireshark.org/lists/wireshark-dev
Unsubscribe: https://wireshark.org/mailman/options/wireshark-dev
 mailto:wireshark-dev-requ...@wireshark.org?subject=unsubscribe

Re: [Wireshark-dev] [Wireshark-commits] rev 52701: /trunk/epan/ /trunk/epan/: oids_test.c

2013-10-21 Thread Ed Beroset
Joerg Mayer wrote:
On Sun, Oct 20, 2013 at 02:18:19AM +, eapa...@wireshark.org wrote:
 http://anonsvn.wireshark.org/viewvc/viewvc.cgi?view=revrevision=52701
 
 User: eapache
 Date: 2013/10/20 02:18 AM
 
 Log:
  Don't use g_assert_cmpint, it isn't happy on Windows. g_assert is nearly as 
 good
  except it doesn't produce as nice error messages.

How about adding it to check api?

That's a good idea, since it was certainly news to me. I also looked and 
couldn't find anything in the gnome bugzilla.  The closest I could find was 
this: https://bugzilla.gnome.org/show_bug.cgi?id=516916

So for those of us without handy access to Windows (at the moment) what's the 
issue, exactly?  Are there other parts of gnome testing environment that won't 
work under Windows?

Ed


___
Sent via:Wireshark-dev mailing list wireshark-dev@wireshark.org
Archives:http://www.wireshark.org/lists/wireshark-dev
Unsubscribe: https://wireshark.org/mailman/options/wireshark-dev
 mailto:wireshark-dev-requ...@wireshark.org?subject=unsubscribe


Re: [Wireshark-dev] [Wireshark-commits] rev 52701: /trunk/epan/ /trunk/epan/: oids_test.c

2013-10-21 Thread Evan Huus
 On Oct 21, 2013, at 4:51 PM, Ed Beroset bero...@mindspring.com wrote:
 
 Joerg Mayer wrote:
 On Sun, Oct 20, 2013 at 02:18:19AM +, eapa...@wireshark.org wrote:
 http://anonsvn.wireshark.org/viewvc/viewvc.cgi?view=revrevision=52701
 
 User: eapache
 Date: 2013/10/20 02:18 AM
 
 Log:
 Don't use g_assert_cmpint, it isn't happy on Windows. g_assert is nearly as 
 good
 except it doesn't produce as nice error messages.
 
 How about adding it to check api?
 
 That's a good idea, since it was certainly news to me. I also looked and 
 couldn't find anything in the gnome bugzilla.  The closest I could find was 
 this: https://bugzilla.gnome.org/show_bug.cgi?id=516916
 
 So for those of us without handy access to Windows (at the moment) what's the 
 issue, exactly?  Are there other parts of gnome testing environment that 
 won't work under Windows?

It produces funny compile warnings about loss of data casting 64-bit types to 
32 bits. These are currently turned into errors in our build system. I didn't 
really investigated further, as I also don't have easy access to a Windows 
builder.

Evan
___
Sent via:Wireshark-dev mailing list wireshark-dev@wireshark.org
Archives:http://www.wireshark.org/lists/wireshark-dev
Unsubscribe: https://wireshark.org/mailman/options/wireshark-dev
 mailto:wireshark-dev-requ...@wireshark.org?subject=unsubscribe