gnokii-users
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [PATCH 3/8] Remove some unnecessary #ifndef WIN32


From: Pawel Kot
Subject: Re: [PATCH 3/8] Remove some unnecessary #ifndef WIN32
Date: Mon, 3 Dec 2018 18:15:07 +0300

Hi,

On Mon, Dec 3, 2018 at 12:41 PM Ladislav Michl <address@hidden> wrote:
> diff --git a/gnokii/gnokii-calendar.c b/gnokii/gnokii-calendar.c
> index e32bf769..10236bae 100644
> --- a/gnokii/gnokii-calendar.c
> +++ b/gnokii/gnokii-calendar.c
> @@ -264,7 +264,6 @@ gn_error writecalendarnote(int argc, char *argv[], gn_data *data, struct gn_stat
> -#ifndef WIN32
>                 if (error == GN_ERR_NOTIMPLEMENTED) {
>                         switch (gn_vcal_file_event_read(optarg, &calnote, i)) {

Probably excluding it for all WIN32 platforms is excessive, but gn_vcal_file_event_read() is generated by flex, which is missing on native Windows platforms. I'm fine with changing it but it will fail on Visual Studio platforms.

> diff --git a/gnokii/gnokii-monitor.c b/gnokii/gnokii-monitor.c
> index 7fde06c5..30aba457 100644
> --- a/gnokii/gnokii-monitor.c
> +++ b/gnokii/gnokii-monitor.c
> @@ -109,8 +109,6 @@ static gn_error readcbmessage(gn_cb_message *message)
>
>  static void displaycall(int call_id)
>  {
> -/* FIXME!!! */
> -#ifndef WIN32
[...]
>         struct timeval now, delta;
[...]
> -#endif

I believe it was due to some weirdness of struct timeval on Windows. Can you please tell on which compiling environments was it tested?

Cheers,
Paweł
--
Pawel Kot

reply via email to

[Prev in Thread] Current Thread [Next in Thread]