Developer forums (C::B DEVELOPMENT STRICTLY!) > Plugins development

OnGetNetPage vs OnLeave in New Project wizard

<< < (3/4) > >>

oBFusCATed:
As a start.
Can you make a patch where you carefully fix the problem with "base is now a keyword" and commit it? You're doing it for at least a second time, so I guess it doesn't hurt if we fix it in current version of squirrel.

But be careful, I don't think the current commit is correct. And do just a single thing in this commit.

BlueHazzard:

--- Quote from: oBFusCATed on November 22, 2018, 08:31:43 am ---But be careful, I don't think the current commit is correct. And do just a single thing in this commit.

--- End quote ---
Doh.... i was to fistrated yesterday from debugging the issue and made a quick search and replace... This was a BAD idea. Thank you for pointing it out...

BlueHazzard:
Ok, this should be ok for the "base" keyword in the default scripts.
Can someone check this?

oBFusCATed:
I don't like using just different case.
It is pretty easy to make mistakes.
Isn't target more appropriate in most cases?

BlueHazzard:
You are right...
here it is

Navigation

[0] Message Index

[#] Next page

[*] Previous page

Go to full version