Re: Another 1.2.3 candidate bug - #1838

On Mon, 2007-02-12 at 13:21 -0800, Erik Hovland wrote:
> On Tue, Feb 13, 2007 at 12:06:07AM +0300, Dmitriy Korovkin wrote:
> > Erik,
> > While this is a good idea, but the patch actually must be accompanied by
> > the other one that allows setting up these parameters in the OPIE
> > Bluetooth Manager. Before the second part of the work has not been done,
> > this patch IMHO, should not be applied.
> 
> Are you telling me that the 'other one' does not yet exist. So the
> feature is incomplete. This patch should work for any familiar supported
> handheld. And familiar already sets up the sysconfig file for the
> graphical environment.

I think Dmitriy's point is that there are two places in Opie that
handle this sort of thing, one is the bluetooth applet, the other
is the bluetooth manager.

The patch only covers the bluetooth applet. To keep things consistent
a similar patch would be required for the bluetooth manager, or
(preferrably) the code in question would need to be factored out into
a tiny lib to be used by both applications.

Regards,

Rene

-- 
Rene Wagner
rw at handhelds dot org
4F33 7FD7 93B3 166B BADA  D6F8 71A1 FEA8 58B4 36D0

_______________________________________________

http://opie.handhelds.org/cgi-bin/moin.cgi/DeveloperWikiIndex

Opie-devel mailing list
[email protected]
https://handhelds.org/mailman/listinfo/opie-devel

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

iD8DBQBF0PcHcaH+qFi0NtARAhCpAJ9GecQHlblsjJb0Ez2L0EaI9MRpyQCeMZgz
sFIgCMlsRjJTwbL7h+ETkt8=
=htBH
-----END PGP SIGNATURE-----