Cleanup the font driver a successful font load.
I agree. I was puzzled why the need to left the file open, but the cleanup into the fileopen driver function doesn't hurt, so I propose apply only the loadfont.c change, and we doesn't break a posible user program that rely in this 'feature' of the font
At first I also patched loadfont.c only, but if some user program relies on this 'feature' (I wonder how - obtain a driver pointer and then access it's symbols directly?) , it's exactly the loadfont change that will break it. After loading a font, all the driver functionality can be used via the respective high-level GRX functions, so I prefer removing openfile cleanups. The choice is yours, of course, you are the maintainer, and you're right that leaving them won't hurt.
E-gards: Jimmy