Thanks. I should have realized that the 'dim c1, f1 as long' is outdated. Thanks for the other tips. Bob On Jan 25, 2010, at 11:20 AM, Brian S wrote: > > On Jan 25, 2010, at 9:24 AM, rckrueger wrote: >> Feedback welcome, please >> >> #if def _FBtoC >> include resources "C2F.nib" // file to be copied to <app>/Contents/Resources/en.lproj >> >> #endif > If always running in FB5 the "#if def _FBtoC #endif is unnecessary. > > >> include "NibToFB.incl" > The include of "NibToFB.incl" is unnecessary( unless several of the fns in it are being used ) in this case because the function used from it ( fn MakeIntoFBWindow ) is already in the FB5 runtime. > > Just replace the include with: toolbox MakeIntoFBWindow(WindowRef w, SInt32 wNum) to make the runtime version available to your code. > >> dim c1, f1 as long > This syntax is error prone and the compiler defaults for you( and provides a warning ). What you want is 'dim as long c1, f1' > As coded, c1 will be only 2 bytes instead of 4 bytes. This could cause overflow problems and bugs. Don't forget to look at the warnings > > Brian S > > -- > To unsubscribe, send ANY message to: futurebasic-unsubscribe@... >