Re: CVS commit: src/sys/dev/gpio

2015-10-16 Thread Paul Goyette

Shouldn't we also add a dependency from the gpio module to require the
sysmon_taskq module?


On Thu, 15 Oct 2015, Jared D. McNeill wrote:


Module Name:src
Committed By:   jmcneill
Date:   Thu Oct 15 09:07:49 UTC 2015

Modified Files:
src/sys/dev/gpio: files.gpio

Log Message:
pull in sysmon_taskq


To generate a diff of this commit:
cvs rdiff -u -r1.13 -r1.14 src/sys/dev/gpio/files.gpio

Please note that diffs are not public domain; they are subject to the
copyright notices on the relevant files.




+--+--+-+
| Paul Goyette | PGP Key fingerprint: | E-mail addresses:   |
| (Retired)| FA29 0E3B 35AF E8AE 6651 | paul at whooppee.com|
| Kernel Developer | 0786 F758 55DE 53BA 7731 | pgoyette at netbsd.org  |
+--+--+-+


Re: CVS commit: src/usr.bin/make

2015-10-16 Thread Tobias Nygren
On Mon, 12 Oct 2015 16:48:13 +
"Simon J. Gerraty"  wrote:

> Module Name:  src
> Committed By: sjg
> Date: Mon Oct 12 16:48:13 UTC 2015
> 
> Modified Files:
>   src/usr.bin/make: var.c
> 
> Log Message:
> The conditional expressions used with ':?' can be expensive
> eg. exists() does stat(2).
> If 'wantit' is FALSE, we are going to discard everything anyway,
> so skip evaluating the conditional and expanding either lhs or rhs.

/work/src/usr.bin/make/var.c:2772:7: error: variable 'emsg' is used 
uninitialized whenever 'if' condition is false 
[-Werror,-Wsometimes-uninitialized]
  if (wantit)
/work/src/usr.bin/make/var.c:2777:7: note: uninitialized use occurs here
  if (emsg)
/work/src/usr.bin/make/var.c:2772:3: note: remove the 'if' if its condition is 
always true
  if (wantit)
/work/src/usr.bin/make/var.c:2760:19: note: initialize the variable 'emsg' to 
silence this warning
  const char *emsg;
1 error generated.


Re: CVS commit: src/sys/dev/gpio

2015-10-16 Thread Jared McNeill
I should have been more clear in the commit message -- I added the 
dependency on gpiobutton(4), not gpio(4).


On Fri, 16 Oct 2015, Paul Goyette wrote:


Date: Fri, 16 Oct 2015 15:47:43 +0800 (PHT)
From: Paul Goyette 
Reply-To: p...@whooppee.com
To: source-changes-d@NetBSD.org
Cc: Jared D. McNeill 
Subject: Re: CVS commit: src/sys/dev/gpio

Shouldn't we also add a dependency from the gpio module to require the
sysmon_taskq module?


On Thu, 15 Oct 2015, Jared D. McNeill wrote:


Module Name:src
Committed By:   jmcneill
Date:   Thu Oct 15 09:07:49 UTC 2015

Modified Files:
src/sys/dev/gpio: files.gpio

Log Message:
pull in sysmon_taskq


To generate a diff of this commit:
cvs rdiff -u -r1.13 -r1.14 src/sys/dev/gpio/files.gpio

Please note that diffs are not public domain; they are subject to the
copyright notices on the relevant files.




+--+--+-+
| Paul Goyette | PGP Key fingerprint: | E-mail addresses:   |
| (Retired)| FA29 0E3B 35AF E8AE 6651 | paul at whooppee.com|
| Kernel Developer | 0786 F758 55DE 53BA 7731 | pgoyette at netbsd.org  |
+--+--+-+