On 15 November 2024 at 01:11 +01, Randy Taylor <dev@rjt.dev> wrote:
> […] the patch didn't apply cleanly for me […]

Here is an updated version of the patch against today's master branch.

> […] If you do the same as treesit-font-lock-level, […]

For some reason it did not work in the past but it does now, so that's how I proceeded.

> You will also need to: (require 'rust-ts-mode)

Ok — some tests do that, others don't, so I did not originally change that.  I've now added the “require”.

> After you make that change, I think you should be able to get rid of
> this:
> +(put 'rust-ts-mode-fontify-number-suffix-as-type 'safe-local-variable
> +     'booleanp)

I first thought that one may want to set that change for specific file but, you are right, it is more of a user choice that should be set in your own config. files.

> In the font-lock-number.rs file, it looks like there is an extraneous
> newline at the top of the file.

Fixed.

> BTW the current patch still doesn't apply against the latest master
> for me.

I have rebased it on the current origin/master.

Best,
C.