--Vr2UxLU0KdcKBaxP
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
Content-Transfer-Encoding: quoted-printable

On Sun, Aug 06, 2006 at 08:53:12PM +0200, Quentin Garnier wrote:
> > >  - you don't free freq_names if k8pnow_current_state is NULL (what if
> > >    the PSB doesn't list the current state?  you know vendors...  I
> > >    certainly expect one to ship a laptop with a broken PSB but a corr=
ect
> > >    ACPI table)
> >=20
> > Fixed.
>=20
> No.  As I said, Coverity will tell you.

Just a side comment..  regardless of the rest of the details, it is
unfair to criticise Juan for not getting code review, while at the
same time providing feedback that is, I have to say, cryptic at best.

If you can see a specific problem with the code, and especially if the
author seems to have missed your point last time, please just describe
the problem clearly.

Peace, Love, and XML.

--
Dan.
--Vr2UxLU0KdcKBaxP
Content-Type: application/pgp-signature
Content-Disposition: inline

-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.4 (NetBSD)

iD8DBQFE1m/GEAVxvV4N66cRAuPEAKDIdBAUA3AS0YLLjLGagTvLe9vI/gCfWgrB
22mEG98rYOE4FRph05nNneU=
=rSZj
-----END PGP SIGNATURE-----

--Vr2UxLU0KdcKBaxP--