great, thanx Phil! glad you're in holidays and back for more :)
Welcome back Karel, i'm glad you're ok and I hope you'll stay ok for a
long time :) ( I didn't understand your fix on nudge btw, but if you say
it works, kudos!)
Phil, you changed capture.c to return 0 if the setting doesn't exist..
do
you think we should do it that way ? I'm not sure... maybe you're
right...
I don't have time to analyze the consequences but take a few minutes to
think of any side-effect of this.. thankx
Also, I though of this (first is your code, second is my proposition) :
! if { [info exists ::grabbers($grabber)] } {
! set windows $::grabbers($grabber)
! } else {
! set windows [list]
! }
!
! lappend windows $window
! set ::grabbers($grabber) $windows
how about
! if { ![info exists ::grabbers($grabber)] } {
! set $::grabbers($grabber) [list]
! }
!
! lappend ::grabbers($grabber) $window
isn't it the same ? but clearer, nicer?
good job Phil, if you're really just resting now, it would be nice to go
through all the bug reports in amsn.sf.net/bugs and fix as many as
possible...
KKRT
On Mon, 19 Dec 2005 11:41:31 -0500, Le Philousophe - Phil
<[EMAIL PROTECTED]> wrote:
> Hehe,
> but who said it isn't fixed ??? :p
> I fixed it today !!
> Phil
>
> Le Monday 19 December 2005 17:29, Karel Demeyer a écrit :
>> I found another bug with webcamshooter. I'm sorry if I'm the one who
>> introduced it. When you open it and click the properties button and
>> then you click OK in the properties window it stops the grabber ...
>> which it shouldn't. If this doesn't get fixed before 0.95 we should
>> state somewhere it's a known bug so we don't et too much reports
about
>> it.
>>
>> friendly greeting,
>>
>> Karel.
>>
>> Op ma, 19-12-2005 te 17:17 +0100, schreef Le Philousophe - Phil:
>> > Hi,
>> > Fixed. Capture-nong wasn't affected by the bug.
>> > I hope this time is the good !! I replaced some "" by lists and
it's
>> > easier to read !! :p
>> > Phil
>> >
>> > Le Monday 19 December 2005 04:50, Youness Alaoui a écrit :
>> > > Hey,
>> > > we're still getting a lot of bugs about the webcam, there are a
few
>> > > things to be fixed before the release (officialy the new package
>>
>> will
>>
>> > > be labeled as of december 21th...)
>> > > Phil, you did a if {[string is digit -strict $...] } {
>> > >
>> > > ::Capture::Set... } .. wel, you should also do an if { [string is
>>
>> digit
>>
>> > > -strict $...] } { $slides.. set $...}
>> > > because for example, the 'Hue' setting is usually inexistent in
most
>> > > webcams and we get an empty string from the GetHue procedure
(since
>>
>> it
>>
>> > > can't find the value, it doesn't set anything in the
interpreter...
>>
>> not
>>
>> > > 0, just empty string)... we could probably also fix it with an
else
>>
>> {
>>
>> > > Tcl_SetObjResult (... 0); } in capture.c .. (btw, is
caputre-nong.c
>> > > updated at the same time ?)
>> > > anyways, I would suggest we check for the value before setting
the
>> > > slides... we could even do an else { $slide_... configure -state
>> > > disabled } ok ?
>> > > Also, the same modifications should be added in the WebcamShooter
>> > > plugin! Also, the default value should be set back to ":::" NOT
>> > > "0:0:0:0" or else, new users will get a webcam configured at 0 0
0 0
>> > > for the first time which makes it into a full black image!!
>> > >
>> > > In the same way, we also have a bug where you press cancel in the
>> > > properties page...
>> > > here's the bug report :
>> > > http://amsn.sourceforge.net/bugs/admin/index.php?bug=1601#
>> > >
>> > > there you can clearly see that each value should be quotes OR
use a
>> > > [list] instead of "" for the command to execute... (ie. button
>>
>> -command
>>
>> > > [list
>> > >
>> > > ::propCancel $b $h $c $whatever] )
>> > >
>> > > go through amsn.sf.net/bugs .. you'll see plenty of things to
fix..
>> > > whoever has time, please go through these and fix as many as
>> > > possible... AS LONG AS WE HAVE SAFE FIXES THAT MAY NOT BREAK
>>
>> ANYTHING!
>>
>> > > Thanks,
>> >
>> > -------------------------------------------------------
>> > This SF.net email is sponsored by: Splunk Inc. Do you grep through
log
>> > files for problems? Stop! Download the new AJAX search engine
that
>> > makes searching your log files as easy as surfing the web.
DOWNLOAD
>> > SPLUNK! http://ads.osdn.com/?ad_idv37&alloc_id865&op=click
>> > _______________________________________________
>> > Amsn-devel mailing list
>> > Amsn-devel@lists.sourceforge.net
>> > https://lists.sourceforge.net/lists/listinfo/amsn-devel
>>
>> -------------------------------------------------------
>> This SF.net email is sponsored by: Splunk Inc. Do you grep through
log
>> files for problems? Stop! Download the new AJAX search engine that
>> makes
>> searching your log files as easy as surfing the web. DOWNLOAD
SPLUNK!
>> http://ads.osdn.com/?ad_idv37&alloc_id865&op=Click
>> _______________________________________________
>> Amsn-devel mailing list
>> Amsn-devel@lists.sourceforge.net
>> https://lists.sourceforge.net/lists/listinfo/amsn-devel
>
> -------------------------------------------------------
> This SF.net email is sponsored by: Splunk Inc. Do you grep through log
> files
> for problems? Stop! Download the new AJAX search engine that makes
> searching your log files as easy as surfing the web. DOWNLOAD
SPLUNK!
> http://ads.osdn.com/?ad_idv37&alloc_id865&op=click
> _______________________________________________
> Amsn-devel mailing list
> Amsn-devel@lists.sourceforge.net
> https://lists.sourceforge.net/lists/listinfo/amsn-devel
