tag:blogger.com,1999:blog-6213675832373085044.post2403050361156756126..comments2008-07-10T05:43:10.427-07:00Branden Tangahttps://profiles.google.com/105501949381806259169noreply@blogger.comBlogger9125tag:blogger.com,1999:blog-6213675832373085044.post-91064389383230876662008-07-10T05:43:00.000-07:002008-07-10T05:43:00.000-07:00Thanks Jørn and Bart! You guys were right!Branden Tangahttp://www.blogger.com/profile/16583314183612055492noreply@blogger.comtag:blogger.com,1999:blog-6213675832373085044.post-27508741775481228372008-07-10T05:32:00.000-07:002008-07-10T05:32:00.000-07:00duh, stupid I missed that :-) Yes, Jørn is right.<BR/><BR/>As a naming convention you should stick to: TFoo is a class, aFoo is an instance and cFoo is a constant.Bart Roozendaalhttp://www.blogger.com/profile/15409758708073886550noreply@blogger.comtag:blogger.com,1999:blog-6213675832373085044.post-20376223236387019782008-07-10T04:51:00.000-07:002008-07-10T04:51:00.000-07:00It is the case-thing that causes the trouble.<BR/><BR/>When you call Variabel.Create, the (already created) instance of Variabel is Re-Initialized.<BR/><BR/>When you call CParsedFile.Create, Delphi tries to re-initialize the variable cParsedFile. cParsedFile isn't created, you get the Access violation...<BR/><BR/>Se example: http://delphi.about.com/od/adptips2006/qt/init_object.htmJørnhttp://www.blogger.com/profile/17894986324482699893noreply@blogger.comtag:blogger.com,1999:blog-6213675832373085044.post-26262461567273314082008-07-10T04:40:00.000-07:002008-07-10T04:40:00.000-07:00Jorn: Thanks, I realized the case insensitivity as I was posting it. I have too much C# and Java on the brain =). I will also use the "T" in my naming convention in the future.<BR/><BR/>Bart: That makes sense, except how does changing "CParsedFile.Create" to "global.CParsedFile.Create" solve a case where there may be an invalid parameter passed to the constructor?Branden Tangahttp://www.blogger.com/profile/16583314183612055492noreply@blogger.comtag:blogger.com,1999:blog-6213675832373085044.post-22993735758341314742008-07-10T04:38:00.000-07:002008-07-10T04:38:00.000-07:00Maybe you can add the screendumps as an enclosure to the posting?Bart Roozendaalhttp://www.blogger.com/profile/15409758708073886550noreply@blogger.comtag:blogger.com,1999:blog-6213675832373085044.post-23668555150538028592008-07-10T04:33:00.000-07:002008-07-10T04:33:00.000-07:00One thing I noticed that you use a construction like A.B.C[i].foo. There isn't any checking on the validity of B or C[i]. It might well be that either is invalid and the software runs into an execption when using that (which is probably inside the constructor). There would be no way for the compiler to know that this would happen by the way.Bart Roozendaalhttp://www.blogger.com/profile/15409758708073886550noreply@blogger.comtag:blogger.com,1999:blog-6213675832373085044.post-5853171775002270752008-07-10T04:30:00.000-07:002008-07-10T04:30:00.000-07:00Remember that Delphi is NOT case sensitive.<BR/><BR/>That means that cParsedFile equals CParsedFile.<BR/><BR/>By convention you should prefix all types with a "T". Rename your CParsedFile class to TParsedFile.<BR/><BR/>Tip: The easiest way is to select all text in global.pas and hit Ctrl+Shift+J. This triggers the "Sync Edit Mode" witch will rename every CParsedFile synchronously.<BR/><BR/>BTW: Pic 1 is not expandable...Jørn Einarhttp://www.blogger.com/profile/17894986324482699893noreply@blogger.comtag:blogger.com,1999:blog-6213675832373085044.post-76361374697710216492008-07-10T04:29:00.000-07:002008-07-10T04:29:00.000-07:00The first screenshot doesn't pop up.Bart Roozendaalhttp://www.blogger.com/profile/15409758708073886550noreply@blogger.comtag:blogger.com,1999:blog-6213675832373085044.post-75231976912268532602008-07-10T03:47:00.000-07:002008-07-10T03:47:00.000-07:00I hope you guys can see the expanded pics when you click on the thumbnail. Blogger seems kinda wonky at times.Branden Tangahttp://www.blogger.com/profile/16583314183612055492noreply@blogger.com