Nut upsdev - Jun 2011 - [nut-commits] svn commit r3081 - branches/apcsmart-dev/drivers

Citeren Michal Soltys <msoltyspl-guest at alioth.debian.org>:
> @@ -251,14 +251,14 @@
>  	if (do_lock_port) {
>  		errno = 0;
>  		ret = 0;
> -#ifdef HAVE_UU_LOCK
> +		#ifdef HAVE_UU_LOCK
>  		ret = uu_lock(xbasename(device_path));
> -#elif defined(HAVE_FLOCK)
> +		#elif defined(HAVE_FLOCK)
>  		ret = flock(fd, LOCK_EX | LOCK_NB);
> -#elif defined(HAVE_LOCKF)
> +		#elif defined(HAVE_LOCKF)
>  		lseek(fd, 0L, SEEK_SET);
>  		ret = lockf(fd, F_TLOCK, 0L);
> -#endif
> +		#endif
>  		if (ret < 0)
>  			upslog_with_errno(LOG_ERR, "apc_ser_try: couldn't lock the
port
> (%s)", device_path);
>  	}
Trivial - preprocessor directives are *not* C-code and should start in  
column 1 (and not inlined with the code) for clarity. So it was  
actually correct before this commit... :-)

Best regards, Arjen
-- 
Please keep list traffic on the list (off-list replies will be rejected)