Re: [Tkdiff-discuss] tkdiff 4.0b1 available

[Dorothy R. <mo...@ar...>]

I found a mistake which caused filenames loaded from the Browse dialog - =
ie., on=20
a PC - to not withstand spaces in the filenames.  I fixed it, along with =
the=20
setcolor problem, and submitted a patch on Sourceforge.  (I'm sure the up=
load=20
"took" this time.)

On another subject - I want to allow tkdiff to do a cvs rdiff like
"cvs -d repository rdiff -r1.1 -r1.2 modulepath/filename" on a file that =
isn't=20
checked out.  Right now it won't do that, because it doesn't know from rd=
iff and=20
because it checks to see if the file given in the argument exists, which =
it=20
won't in this case.  So I thought of adding a command line option for doi=
ng an=20
rdiff, and removing the check for the existence of the file.  Something l=
ike
"tkdiff -d repository -rdiff -r1.1 -r1.2 modulepath/filename"
Objections?  Better ideas?  I can't help thinking there might be an easie=
r way.

Dorothy

Daniel H=F6pfl wrote:
> Hi!
>=20
> In line 153-158 you have:
>=20
> ------------------------------------------------------
> set w(selcolor) [lindex [.testent configure -selectbackground] 3]
> set entryfont [lindex [.testent configure -font] 3]
> destroy .testent
> # the above results in a nearly undistinguishable darker gray for the
> # selected color (rh8 with tk 8.3.3-74) "#c3c3c3"
> set w(selcolor) "#b03060"
> ------------------------------------------------------
>=20
> I think the problem is the index 3 when setting w(selcolor):
>=20
> ------------------------------------------------------
> set w(selcolor) [lindex [.testent configure -selectbackground] 4]
> set entryfont [lindex [.testent configure -font] 3]
> destroy .testent
> ------------------------------------------------------
>=20
> ... sets the value of "*selectBackground" from your .Xdefaults (or=20
> #bfdfff, a light blue, as default) into w(selcolor).
>=20
> An other solution would be:
>=20
> ------------------------------------------------------
> set w(selcolor) [lindex [.testent configure -selectforeground] 4]
> set entryfont [lindex [.testent configure -font] 3]
> destroy .testent
> ------------------------------------------------------
>=20
> In this case you get the value of "*selectForeground". Better than usin=
g=20
> the background color for the foreground of a checkbox.
>=20
> Is there a way to get the value of "*selector" from the .Xdefaults? I=20
> think this would be the "correct" value for this case.
>=20
> Greetings,
>    Daniel
>=20

--=20
-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=
=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-
Dorothy Robinson
http://www.twobarleycorns.net