Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

[example.pl] add example.pl #640

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from
Draft

[example.pl] add example.pl #640

wants to merge 1 commit into from

Conversation

bw1
Copy link
Member

@bw1 bw1 commented Jun 7, 2019

No description provided.

@bw1 bw1 added the example label Jan 6, 2020
@ailin-nemui ailin-nemui marked this pull request as draft April 22, 2020 18:11

Irssi::signal_add('setup changed', \&sig_setup_changed);

Irssi::settings_add_str($IRSSI{name} ,$IRSSI{name}.'_test_str', 'hello world!');
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

shouldn't the 'hello world!' be $test_str instead, and the default set by initialising this variable?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

settings_add_str let add irssi a setting with default value 'hello world!'

`sub sig_setup_changed' read out settings and maybe check values or react on chance of value

$test_str is for working in script (value is checked and change is already done)
init of this scalar is on last line

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hah, makes sense. I never stop learning, and I think I got it now. So please disregard my comment.

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants