Gammon Forum : MUSHclient : General : Lets start the conversion.

Register forum user name Search FAQ

Gammon Forum

Entire forum

MUSHclient

General

Lets start the conversion.

Lets start the conversion.

It is now over 60 days since the last post. This thread is closed. Refresh page

Pages: 1 2 3 4 5

Posted by

David Berthiaume (202 posts)

Date

Reply #45 on Mon 13 Dec 2004 10:40 PM (UTC)

Message

Quote:
I have added a couple of new things to MUSHclient 3.60. The first one lets you do a script just before MUSHclient saves its world to disk. This is a logical spot to serialize from internal variables to the external variables.

yeah, I used to have a timer that did that every 5 minutes or so, but I don't remember the exact syntax... Not to mention when, I re-scripted it after my computer meltdown, it didn't save in the spot that I wanted it too, it kept saving in C:\My Documents, which I didn't want, I wanted it saved in D:\Program Files\MUSHclient\worlds

Anyways... I'll figure that out eventually.

Posted by

Nick Gammon Australia (22,975 posts)

bio Forum Administrator

Date

Reply #46 on Mon 13 Dec 2004 11:01 PM (UTC)

Message

A save with no filename should save in the default place (ie. the current file).

eg.

Save ""

- Nick Gammon

www.gammon.com.au, www.mushclient.com

Posted by

Flannel USA (1,230 posts)

Date

Reply #47 on Mon 13 Dec 2004 11:03 PM (UTC)

Message

I think you can use .. "|%1"
But if that doesnt work, you can probably also use .. "|" .. "%1"

And then you have expand variables checked, and (@replacetext) in your match text.

~Flannel

Messiah of Rose
Eternity's Trials.

Clones are people two.

Posted by David Berthiaume (202 posts) bio

Date

Reply #48 on Mon 13 Dec 2004 11:07 PM (UTC)

Amended on Mon 13 Dec 2004 11:11 PM (UTC) by David Berthiaume

Message

The only issue I see with that is a possible nil return when there is no variable there in the first place.

run it through a script(you're gonna be scripting in it already to set the table element):

if GetVariable ("replacetext") == nil then
SetVariable ("replacetext", "%1")
else
SetVariable ("replacetext", "@replacetext" .. "|" .. "%1")
end -- if

On a side note, I was just thinking of my infobar script, I could do away with variables completely, I think. Use table's to store the information, nothing would really change, but instead of saving to the variable list, it would be updating the table... Then along with all my other tables, when I save the world, or whatever, it'll save that into a single variable, which will reload itself aferwards. Man, I like this table thing... I never understood it in VBscript before, but Lua seems pretty straight forward in how the tables work.

Posted by Nick Gammon Australia (22,975 posts) bio Forum Administrator

Date Reply #49 on Mon 13 Dec 2004 11:20 PM (UTC)

Message

You can make that a bit shorter. For one thing testing for not-nil can be done without the == nil bit.

Second, if you use the "var" table you can save the GetVariable/SetVariable stuff. So it could read:


if var.replacetext then
  var.replacetext = var.replacetext .. "|%1"
else
  var.replacetext = "%1"
end -- if

- Nick Gammon

www.gammon.com.au, www.mushclient.com

Posted by Nick Gammon Australia (22,975 posts) bio Forum Administrator

Date Reply #50 on Mon 13 Dec 2004 11:22 PM (UTC)

Message

In many ways it is easier to stick with tables anyway, and use table.concat to put in the "|" symbols. Here is an example:


t = { "fish", "chips", "cat", "dog" }

print (table.concat (t, "|"))

-- Ouput: fish|chips|cat|dog

- Nick Gammon

www.gammon.com.au, www.mushclient.com

Posted by

David Berthiaume (202 posts)

Date

Reply #51 on Mon 13 Dec 2004 11:27 PM (UTC)

Message

Yeah, that's true, it would be easier to stick with tables, but that doesn't change the fact that I need a regexp trigger to match against the table...

That reminds me, if we get the replace text thing working perfectly, then they'll be one last issue, it's CAPS sensitive. even if you're matching ignore case, it won't work. I tested that earlier when I first started playing with it.

Posted by

Nick Gammon Australia (22,975 posts)

bio Forum Administrator

Date

Reply #52 on Tue 14 Dec 2004 12:04 AM (UTC)

Amended on Tue 14 Dec 2004 02:06 AM (UTC) by Nick Gammon

Message

In what context are you doing the test? Triggers can be made case-insensitive. So can regexps.

The output from that table concatenation (fish|chips|cat|dog) can be used in a regexp.

- Nick Gammon

www.gammon.com.au, www.mushclient.com

Posted by

Flannel USA (1,230 posts)

Date

Reply #53 on Tue 14 Dec 2004 12:13 AM (UTC)

Amended on Tue 14 Dec 2004 12:21 AM (UTC) by Flannel

Message

Why would the variable be empty to begin with?

and I dont see how sticking with tables would help.

Youd need to have a table to keep your targets, as well as a MC variable.

Edit:
I see, youre using the table to store the information for the switch. So you add to the table, and then regenerate your MC variable. Wouldnt the output be both columns though?

As for a long term storage solution, which could just be a MC variable for each thing (name, and replacement names), or you can store it in a database.

~Flannel

Messiah of Rose
Eternity's Trials.

Clones are people two.

Posted by David Berthiaume (202 posts) bio

Date Reply #54 on Tue 14 Dec 2004 12:22 AM (UTC)

Message

All right:

function textreplace (name, line, wildcards, styles)

  for _, v in pairs (styles) do
    
    ColourTell (RGBColourToName (v.textcolour), 
                RGBColourToName (v.backcolour), 
                (string.gsub (v.text, "Altair", "Dummy")))  
  end

end

This is the case sensitive part:

(string.gsub(v.text, "Altair", "Dummy")))

<triggers>
<trigger
enabled="y"
ignore_case="y"
keep_evaluating="y"
match="Altair"
omit_from_output="y"
regexp="y"
repeat="y"
script="textreplace"
sequence="100"
>
</trigger>
</triggers>

We have success, this works, Altair(script version) == Altair(Trigger Version)

Ok, lets change it now. Now we match on "altair"

altair(Trigger with ignore case) does not match Altair in the script

Flip it around again, Make the Trigger Altair, and the script altair, it doesn't match up.

Now if you match on altair, and you type say altair, then it will work, if you type: say Altair, no dice, it won't work...

So what I mean is that the code that you put up is case senstive, even thought the trigger says it's not supposed to be... That might be a slight bug... Perhaps. I don't know.

Posted by

Flannel USA (1,230 posts)

Date

Reply #55 on Tue 14 Dec 2004 12:30 AM (UTC)

Message

You can check the "convert wildcards to lowercase" OR you can just cast the name (wildcard, once its passed) to lowercase while you're checking.

Then you'll have to have your match table be all lower case (you convert to lowercase (.lower) when you add it).

~Flannel

Messiah of Rose
Eternity's Trials.

Clones are people two.

Posted by

David Berthiaume (202 posts)

Date

Reply #56 on Tue 14 Dec 2004 12:34 AM (UTC)

Message

yeah, converting to lowercase works, but annoying when you have to script AddTrigger... *shrugs* Oh well, such is life.

I'm still trying to work out how to combine the first replace text script, and the second example Nick gave us.

I should probably stop for the night, go to bed, and think about it tommorrow morning. when I'm not brain dead tired.

Posted by

Flannel USA (1,230 posts)

Date

Reply #57 on Tue 14 Dec 2004 12:51 AM (UTC)

Message

Why are you scripting addtrigger?

Thats why were using the variable, so you can match on anyone inside your list, and then convert. All in one trigger.

~Flannel

Messiah of Rose
Eternity's Trials.

Clones are people two.

Posted by

David Berthiaume (202 posts)

Date

Reply #58 on Tue 14 Dec 2004 01:09 AM (UTC)

Message

Ok, so you set the variable... how do you match lines of the mud to a variable?

You have to use a trigger.

That's why you have to script an AddTriger

AddTrigger("Textreplace", var.replacetext , "", trigger_flag.Enabled , custom_colour.Custom15, 0, "", "")

make it regexp and lowercase wilds, and what not. Anyways, you have to keep replacing the trigger that matches the replacetext that you want. Otherwise setting up the variable is useless, and like Nick said, we don't even need the variable really, we can script it right into a table and that will speed things up.

Posted by

Flannel USA (1,230 posts)

Date

Reply #59 on Tue 14 Dec 2004 01:22 AM (UTC)

Message

But then you have a trigger for each name.
Which you might as well ditch the table for, and hard code it directly into the trigger (no more lookup, just a subroutine which replaces X with Y).

You can have a trigger match ANY name in your table (which is also in your variable) using expand variables and a variable-regexp. So the same trigger matches constantly, and you dont use AddTrigger anywhere.

Oh, youre hard-coding the variable into the trigger. The problem with that is then you have to add the parenthesis to the outsides of the list. Which, is doable.

You would lose the information about the trigger (number of matches and such) which may or may not be an issue.

And I'd imagine setting the variable takes less time than adding a trigger. You'd need to set the MC variable eventually to save your list between sessions (or use a database, but I think a database might be overkill).

~Flannel

Messiah of Rose
Eternity's Trials.

Clones are people two.

The dates and times for posts above are shown in Universal Co-ordinated Time (UTC).

To show them in your local time you can join the forum, and then set the 'time correction' field in your profile to the number of hours difference between your location and UTC time.

127,336 views.

This is page 4, subject is 5 pages long: 1 2 3 4 5

It is now over 60 days since the last post. This thread is closed. Refresh page

Go to topic: Search the forum

Quick links: MUSHclient. MUSHclient help. Forum shortcuts. Posting templates. Lua modules. Lua documentation.

Information and images on this site are licensed under the Creative Commons Attribution 3.0 Australia License unless stated otherwise.

Comments to: Gammon Software support
Forum RSS feed ( https://gammon.com.au/rss/forum.xml )