Jim,
your practice seems fine.
In fact, renaming your include file from .inc to .asp is a great practice. If someone where to call your include file from the browser, they would be able to download the ".inc" version, but the ".asp" version would be parsed. This will prevent outside users from viewing your source.
I wouldn't worry about anything. Your code seems to be fine.
__________________
Shane Bauer
Microsoft Certified Professional (MCP) - ASP.NET
ASP/ASP.net, C#, VB/VB.NET, PHP, Perl, SQL
|