GoScintilla for PB 5.x

[Tristano]

GoScintilla for PB >= 5.10

Just wanted to inform everyone that I’ve published an updated version of Stephen’s GoScintilla, compatible with PureBASIC 5.x:

• GoScintilla v3 at PureBASIC Archives

• The changes required to make it compatible with PB 5.x brake its backward compatibility with PureBASIC versions < 5.10.
• For this reason I've changed its version number to 3.0.0 (following Semantic Version 2.0 standard, which is customary on GitHub).
• No new features added, just fixed GoScintilla.pbi to work with PB 5.x.

I want to thank Stephen (srod) for permission to go ahead with this project.

• GoScintilla original post on PB Forums

All contributions are welcome.

B.R.

Tristano

Post Sscriptum

After consulting with GoScintilla's author, I wanted to emphasize that version 3 in my fork of GoScintilla only refers to the fact that it breaks backward compatiblity.
IT DOESN'T MEAN that it adds new features — it doesn't (not yet at least).

I've changed its MAJOR version number because on GitHub is customary to follow Semantic Versioning 2.0 standard:

Major version X (X.y.z | X > 0) MUST be incremented if any backwards incompatible changes are introduced to the public API. It MAY include minor and patch level changes. Patch and minor version MUST be reset to 0 when major version is incremented.

I apologize if this lead to any confusion, I didn't mean to give the impression that I had introduced new features; I only wanted to distribute an updated version that works with current PB 5.x.

I've now also renamed the project folder from "GoScintilla3" to "GoScintilla", to de-emphasize the version number; and now refer to it just as GoScintilla for PB5.x in the documentation. References to its being v3.0 are kept along with explanation why I changed it.

[srod]

Tristano, the current version of GoScintilla (2.7) seems to work fine with PB 5.51 x86. Is there something I am missing?

Otherwise I am happy for you to do what you wish with the code.

[ts-soft]

@srod
I have only tested to first error: #PB_Sort_String

Greetings
Thomas

[srod]

Ah, memory is getting faulty in my old age. I adjusted the code some time ago so that it was compatible with PB 5.5x and plain forgot about it!

Doh!

[Tristano]

Yes, there were 3 lines of code which I had to change for it to work with PB5.x.

Line 108, "Native types can't be used with pointers" (now: *bytePointer):

Code: Select all

        *bytePointer.i              ;Used for left delimiters (separators).

Line 1787, "Constant not found: #PB_Sort_String" (now: #PB_String):

Code: Select all

        SortStructuredList(*this\Keywords(), #PB_Sort_Ascending|t1, OffsetOf(GoScintillaKeyword\keyWord$), #PB_Sort_String)

Line 1853 (same as above):

Code: Select all

       SortStructuredList(*this\Keywords(), #PB_Sort_Ascending|t1, OffsetOf(GoScintillaKeyword\keyWord$), #PB_Sort_String)

Both these errors refer to changes brought in with PureBasic release 5.10:

http://www.purebasic.com/documentation/ ... story.html

14th February 2013 : Version 5.10
- Changed: replaced #PB_Sort_<Type> with #PB_<Type> to avoid possible errors
- Changed: disallow native type for pointers

So, these 3 small changes I've made are making GoScintilla work with PB >= 5.10, and break its compatiblity with PB <5.10.

Hence, I opted to update its version number to 3.0.