By Date: <-- -->
By Thread: <-- -->

[PATCH] Keymap compression and budget-ci IR conversion (update)

I demand that Nickolay V. Shmyrev may or may not have written...

> Darren Salt wrote:
>> I demand that I definitely did write...
>>> The patch set can be downloaded here:
>>> <URL:http://www.youmustbejoking.demon.co.uk/progs/linux/dvb-ir-20060206.patch.tar.gz>
>>> Reference changeset is 3316.
>> I've updated that patch set - part of patch 4 was missing.

> Hi Darren, patch for conversion to ir-common now looks very good. It could
> be applied but the only reason is dependency on other changes.

Right... untangling it from the other patches should be reasonably easy...
I'll follow up with a set of patches which puts the bulk of the budget-ci
changes first.

> I am just wondering, what's the problem with repeat?

I think that I'm happier with the newer implementation in that repeats
*don't* show up if data isn't being received from the remote control,
allowing a longer time-out without auto-repeat causing lots more input than
intended.

> About second part of your changes I have the following question: is there
> any reason to use perl?

Convenience, speed of implementation, maintainability...

> Why not just use plain C code as we was doing before? It will simplify
> makefile process that is why I am asking.

Well, if you want to rewrite the keymap packer in C, that's fine :-)

Distribute the generated source if you like, but don't throw away its source
or the generator unless you really want to have to edit something which is
significantly less maintainable.

-- 
| Darren Salt | nr. Ashington, | d youmustbejoking,demon,co,uk
| Debian,     | Northumberland | s zap,tartarus,org
| RISC OS     | Toon Army      |  (at)                       Say NO to UK ID cards
|                                                       http://www.no2id.net/

PC voice recognition software: "Format C: return". "Yes, return".

--
video4linux-list mailing list
Unsubscribe mailto:video4linux-list-request (at) redhat.com?subject=unsubscribe
https://www.redhat.com/mailman/listinfo/video4linux-list