Closed Bug 31532 Opened 25 years ago Closed 24 years ago

colchange does not check that @::legal_keywords is defined

Categories

(Bugzilla :: Bugzilla-General, defect, P3)

x86
Windows NT
defect

Tracking

()

VERIFIED FIXED
Bugzilla 2.12

People

(Reporter: morpheus, Assigned: Chris.Yeh)

Details

(Whiteboard: 2.12)

Attachments

(1 file)

colchange does not check that @::legal_keywords is defined and barfs.
Attached patch Patch for colchange (deleted) — Splinter Review
tara@tequilarista.org is the new owner of Bugzilla and Bonsai. (For details, see my posting in netscape.public.mozilla.webtools, news://news.mozilla.org/38F5D90D.F40E8C1A%40geocast.com .)
Assignee: terry → tara
Hey Morpheus, This is probably related to http://bugzilla.mozilla.org/show_bug.cgi?id=44622 perhaps a duplicate. The patch you've submitted solves the problem by referencing the variable twice, which is enough to shut up the stricter warnings. Does the patch in 44622 solve your crash as well? -tak
marking as possible 2.12, taking ownership
Assignee: tara → cyeh
Whiteboard: 2.12
patches were checked in relating to 44662, does this fix the problem? if it does, please mark the bug fixed.
I am maintaining a Bugzilla install that runs in Perl 5.6, and I can't duplicate this since the patches in bug 44622 were checked in. I agree that it was the same problem.
closing this bug as fixed.
Status: NEW → RESOLVED
Closed: 24 years ago
Resolution: --- → FIXED
Sorry for the spam, but I needed to be able to query for all of these correctly.
Target Milestone: --- → Bugzilla 2.12
<shrug> VERIFIED. Gerv
Status: RESOLVED → VERIFIED
Moving closed bugs to Bugzilla product
Component: Bugzilla → Bugzilla-General
Product: Webtools → Bugzilla
Version: other → unspecified
QA Contact: matty_is_a_geek → default-qa
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: