Great Circle Associates Majordomo-Users
(November 1998)
 

Indexed By Date: [Previous] [Next] Indexed By Thread: [Previous] [Next]

Subject: Re: MajorCool
From: Bill Houle <bhoule @ sparc . sandiegoca . ncr . com>
Date: Thu, 5 Nov 1998 08:00:44 -0800 (PST)
To: jesper @ maartenson . net (Jesper Maartenson)
Cc: majordomo-users @ GreatCircle . COM
In-reply-to: <Pine.LNX.4.05.9811051045490.26584-100000@gaia.hts2.org> from "Jesper Maartenson" at Nov 5, 98 10:46:10 am

Jesper Maartenson said:
> 
> And the edit-box is empty! This list has about 120 members, and shouldn't
> they be listed in the box? Or do I have to write a whole new list of
> subscribers each time I want to edit the listmembers?

Nope, this is a bug. Fixed in 1.3.2 (but not released yet). Quite
frankly, I'm shocked it hasn't been a problem sooner, since its been
around for quite some time. The weird thing is, its worked for most
people.

In the send_who_form() function, $members is defined as a local
variable within an "if" block. Evaluated outside the block, $members
is naturally empty because it is undefined.

It looks like it exercised a "loophole" (read: bug) in Perl; 5.004
and newer scope correctly and break MjC, while 5.003 and below work
(unexpectedly) fine.

Quick fix- Change:

	if ($sizeaction ne "append") {
		# get members as of the time this snapshot was taken
		local($members) = join("\n",
			&get_who("COOL_TMPDIR/$list.$whotime",
			$prefs{GenListSorted}, $subset));
	} 

To:

	local($members);
	if ($sizeaction ne "append") {
		# get members as of the time this snapshot was taken
		$members = join("\n", 
			&get_who("COOL_TMPDIR/$list.$whotime",
			$prefs{GenListSorted}, $subset));
	} 


I'll have 1.3.2 released shortly, after I finish testing a couple
new features.

--bill




References:
Indexed By Date Previous: Re: MajorCool problem
From: Bill Houle <bhoule@sparc.sandiegoca.ncr.com>
Next: the subscribe word problem
From: Shane <shane@shahrokh.com>
Indexed By Thread Previous: Re: MajorCool
From: Paul Wolstenholme <wolstena@sfu.ca>
Next: Me again...
From: Brent Sims <brent@rmi.net>

Google
 
Search Internet Search www.greatcircle.com