[LinuxPPS] [PATCH] time_pps_readlink()

Udo van den Heuvel udovdh at xs4all.nl
Wed Aug 30 17:45:22 CEST 2006


Rodolfo Giometti wrote:
> On Sun, Aug 27, 2006 at 09:26:34AM +0200, Udo van den Heuvel wrote:
>> If someone can explain Rodolfo's idea to make the patch even smaller,
>> please post!
> 
> Here Rodolfo's idea! ;)

Ah, thanks.

> As you can see I used less ifdef lines and the patch is more readable.

I see. I #ifdeffed around the original code, in your example (using the
extra variable in the struct) we just replace some code.

> Please, let me know what do you think about it.

Looks nice and clean. I was wondering about the ~0 but any integer would
have worked?

> P.S. The patch should be tested since I haven't any NMEA device.

Compiling it athis moment. Will try it out soon.
Maybe Philip can test teh patch as well and/or update the NMEA patches
on his website?

Udo



More information about the LinuxPPS mailing list