jyapayne/nimble
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Fixed FD leak.

Browse source
This commit is contained in:
Yuriy Glukhov 2016-04-06 15:21:55 +03:00
commit 36d09de320
1 changed files with 2 additions and 2 deletions
Download patch file Download diff file Expand all files Collapse all files

4
src/nimblepkg/packageparser.nim
View file

@ -118,6 +118,7 @@ proc readPackageInfoFromNimble(path: string; result: var PackageInfo) =
if fs != nil: if fs != nil:
var p: CfgParser var p: CfgParser
open(p, fs, path) open(p, fs, path)
defer: close(p)
var currentSection = "" var currentSection = ""
while true: while true:
var ev = next(p) var ev = next(p)
@ -172,7 +173,6 @@ proc readPackageInfoFromNimble(path: string; result: var PackageInfo) =
"Invalid package info, should not contain --" & ev.value) "Invalid package info, should not contain --" & ev.value)
of cfgError: of cfgError:
raise newException(NimbleError, "Error parsing .nimble file: " & ev.msg) raise newException(NimbleError, "Error parsing .nimble file: " & ev.msg)
close(p)
else: else:
raise newException(ValueError, "Cannot open package info: " & path) raise newException(ValueError, "Cannot open package info: " & path)
@ -269,4 +269,4 @@ proc getInstalledPkgs*(libsDir: string, options: Options):
raise exc raise exc
proc isNimScript*(nf: string, options: Options): bool = proc isNimScript*(nf: string, options: Options): bool =
result = readPackageInfo(nf, options).isNimScript result = readPackageInfo(nf, options).isNimScript