Review Request 110091: clean up and update FreeBSD support for kinfocenter
Max Brazhnikov
makc at freebsd.org
Sun May 5 12:26:54 BST 2013
> > Ok, here's a patch for GetInfo_ReadfromPipe only:
> > http://people.freebsd.org/~makc/patches/read_from_pipe.diff
>
> Looks good to me (although I have not tested it). Small note: there is
> a contructor of QStringList taking a QString so if you have a list with
> only one item you don't need to construct an empty list and then add
> something using operator<<.
Yes, I'm aware of it, but 'operator<<' looks more clear for me.
> Side-note: there is a comment talking about using /proc/pci if lspci is
> not found. Not that newer kernels do have a /proc/pci at all ;)
I prefer not to make functional changes in (linux) code, which I can't test :)
> >> While you're at it you
> >> can then fix the type (devies -> devices), too ;)
> >
> > I've removed those comments completely.
>
> Another solution for that problem ;)
>
> Eike
Max
More information about the kde-core-devel
mailing list