Skip to content

Conversation

vague666
Copy link
Member

@vague666 vague666 commented Oct 27, 2020

Since the original use case wasn't broken(Fixes #1228) this adds more functionality instead :)

@ailin-nemui
Copy link
Contributor

maybe you forgot to set some variable to NULL?

@vague666
Copy link
Member Author

Only possible var is tmprec, maybe I am remembering wrong that it is NULL when declared?

@ailin-nemui
Copy link
Contributor

now it doesn't crash anymore :-) only the code formatting is left to clean up to confirm to the new standard

@ailin-nemui ailin-nemui changed the title 'Fix' for #1228 Implement priority on the matched lines for /hilight Oct 27, 2020
@vague666
Copy link
Member Author

Maybe I do too much perl and forget that not all languages initialize variables :)

@ailin-nemui ailin-nemui added the auto-merge This PR is scheduled for merge if no further comments are opened label Dec 7, 2020
@ailin-nemui
Copy link
Contributor

@irssi/developers ?

@ailin-nemui ailin-nemui merged commit 581c43e into irssi:master Dec 14, 2020
@ailin-nemui ailin-nemui added this to the 1.3.0 milestone Oct 11, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
auto-merge This PR is scheduled for merge if no further comments are opened
Projects
None yet
Development

Successfully merging this pull request may close these issues.

hilight does not respect -priority arg
2 participants