shravnk:

So you're prompting the user for a filename, and verifying that it contains ".nml" or ".xml". If it doesn't contain that, you print an error message and continue on. Hmmm....

You really want to stop your program when you find a problem, so you should use the die function rather than print on line 8.

Another problem is that with the expression you've given "foo.xml.abc.def" is a valid filename.

After you open the file, you then read the first line of the file, and treat it as a filename to pass to parsefile. Unless your file contains the name of the XML file you're trying to process, that's going to be a bit of a problem...

Also, you have *another* open statement you haven't checked yet...

...roboticus