Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Commit 9e1e149

Browse files
committed
* Lib/sgi/flp.py: fix caching bug (always write the whole file to
the cache!)
1 parent 29e7af0 commit 9e1e149

2 files changed

Lines changed: 8 additions & 28 deletions

File tree

Lib/irix5/flp.py

Lines changed: 4 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -23,22 +23,12 @@
2323
#
2424
def parse_form(filename, formname):
2525
forms = checkcache(filename)
26-
if forms != None:
27-
if forms.has_key(formname):
28-
return forms[formname]
29-
else:
30-
forms = {}
31-
fp = _open_formfile(filename)
32-
nforms = _parse_fd_header(fp)
33-
for i in range(nforms):
34-
form = _parse_fd_form(fp, formname)
35-
if form <> None:
36-
break
26+
if forms is None:
27+
forms = parse_forms(filename)
28+
if forms.has_key(formname):
29+
return forms[formname]
3730
else:
3831
raise error, 'No such form in fd file'
39-
forms[formname] = form
40-
writecache(filename, forms)
41-
return form
4232

4333
#
4434
# Externally visible function. Load all forms.

Lib/plat-irix5/flp.py

Lines changed: 4 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -23,22 +23,12 @@
2323
#
2424
def parse_form(filename, formname):
2525
forms = checkcache(filename)
26-
if forms != None:
27-
if forms.has_key(formname):
28-
return forms[formname]
29-
else:
30-
forms = {}
31-
fp = _open_formfile(filename)
32-
nforms = _parse_fd_header(fp)
33-
for i in range(nforms):
34-
form = _parse_fd_form(fp, formname)
35-
if form <> None:
36-
break
26+
if forms is None:
27+
forms = parse_forms(filename)
28+
if forms.has_key(formname):
29+
return forms[formname]
3730
else:
3831
raise error, 'No such form in fd file'
39-
forms[formname] = form
40-
writecache(filename, forms)
41-
return form
4232

4333
#
4434
# Externally visible function. Load all forms.

0 commit comments

Comments
 (0)