Theme XML files getting borked?

Ekopalypse

copy it over from C:\Program Files (x86)\Notepad++

supasillyass

@supasillyass

copy it over from C:\Program Files (x86)\Notepad++

It’s not there. I’m guessing it gets created at installation. I did, however, try it with the portable version and it does occur:

<LexerType name="actionscript" desc="ActionScript" ext="">
    <WordsStyle name="TYPE WORD" ... >onMotionChanged onMotionFinished Tween ImagesStrip ContentScroller mx transitions easing Sprite Point MouseEvent Event BitmapData Timer TimerEvent addEventListener event x y height width</WordsStyle>

Ekopalypse

@supasillyass

that is correct if one specified user defined keywords in an keyword tag like
in actionscript the “TYPE WORD” is but “COMMENTLINE” is not and
I don’t have any additional word in any of my themes.xml in any of the “COMMENTLINE” tags.

Ekopalypse

@supasillyass

Ok, in addition to the known fact that the themes are not up to date with the default style
some of them seem to have invalid data, meaning having string content in non-keyword tokens.
I do have a FR which, when gets accepted, could be used to cleanup those data as well.

supasillyass

@Ekopalypse said:

@supasillyass

Ok, in addition to the known fact that the themes are not up to date with the default style
some of them seem to have invalid data, meaning having string content in non-keyword tokens.
I do have a FR which, when gets accepted, could be used to cleanup those data as well.

I think this might be the source of the problem, with the invalid data getting propagated across to the different styles. (If I clean up one style XML file directly, the invalid data creeps back in after playing around with the Style Configurator.)

supasillyass

Edit: Bespin.xml, Choco.xml and HotFudgeSundae.xml might be the worst offenders.

Ekopalypse

@supasillyass

Haven’t verified the source code but from what you are telling I assume that npp loads it
on startup and keeps it in memory, meaning, when you stop npp and edit the xml with a
different editor it might not get repopulated.

supasillyass

I tried sanitising the theme files by getting the default set from portable Notepad++ and using grepWin to replace ">.*<\/WordsStyle>" with " \/>". However, the files still get borked because the LexerTypes vary between the themes (ordering also seems to matter). So, when I changed from Obsidian to Black board, my bash and Batch keywords got transferred to TCL and Objective-C (QUALIFIER not INSTRUCTION WORD).

Ekopalypse

Yes, as I’ve mentioned in this FR themes and default style aren’t in sync. So an automate way must
read the default xml and compare tags against the theme files and act accordingly.
Unfortunately @donho hadn’t time to have a look or maybe he has something different in mind, like changing the way themes are currently used.

supasillyass

A kludgy fix is to make the theme XML files read-only and add new keywords directly to langs.xml, using a different text editor. (My langs.xml was pretty old, so I grabbed langs.model.xml from the latest portable version.)