Re: [patch][rfa] Add "active" pin to sid-sched

Hi -

> [...]
> OK to commit?

Looks fine to commit, though please consider not using a new
"active_p" member variable in order to send edges on the active
pin.  This can be done by replacing ...

> + 	  // Drive the active pin if the threshold has been crossed.
> + 	  if (UNLIKELY(! this->active_p))
> + 	    {
> + 	      this->active_pin.drive (1);
> + 	      this->active_p = true;
> + 	    }
> + 

with
	  if (this->active_pin.sense() != 1)
	    this->active_pin->drive(1);

and a corresponding change for the 0 transition.


- FChE

-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.0.7 (GNU/Linux)

iD8DBQE/T3xAVZbdDOm/ZT0RAjwoAJ9XUJtlJ2eIBaU223dw8VaukVF0jQCeJ3Ys
47ZtwQ4CmtdByIe/OpP8wzg=
=bqZ2
-----END PGP SIGNATURE-----