[Other] String concat. crash

[XtrmJosh]

Bug ReportOperating System:
Other Short Description:
String concat. crash Behaviors: Client Freeze/Crash

Indepth Description:
It seems that this code:


local currentTime = os.date("*t", 906000490)
local stamp = currentTime.hour .. " " .. currentTime.min

print(stamp)

Causes the client to crash. Meanwhile, this code:


local currentTime = os.date("*t", 906000490)
local stamp = currentTime.hour .. " : " .. currentTime.min

print(stamp)

Does not. As you can see, the only difference is an extra space and : in the latter.

[Fatality]

They both work for me o.0

[maroxy]

local time = os.date("%X")
print(time)

You can use this one.

[Fatality]

If you want it to return the same as the snippet of code, you could use this

local time = os.date("*t", 906000490)
print('%s %d', time.hour, time.min)

And if you want it as a variable, not just printing it, you can do this.


local time = os.date("*t", 906000490)
local tmp = "%s %s"
local stamp = time1:format(time.hour, time.min)
print(time)

[XtrmJosh]

They both work for me o.0


If you want it to return the same as the snippet of code, you could use this

local time = os.date("*t", 906000490)
print('%s %d', time.hour, time.min)

And if you want it as a variable, not just printing it, you can do this.


local time = os.date("*t", 906000490)
local tmp = "%s %s"
local stamp = time1:format(time.hour, time.min)
print(time)


Thanks - working on finding an exact repro. I tried to simplify too much, might be a Lua implementation issue (e.g one of the XB functions). Investigating, will update shortly.

The exact code, if anyone wants to investigate in tandem:


--[[
This is a XenoBot example script, intended to
teach new users about the scripting API and
act as script that is usable in actual play.

recentPrivateMessageHUD.lua - displays a HUD
showing your most recent private messages.

** DO NOT EDIT THIS FILE. INSTEAD, COPY IT TO
"Documents\XenoBot\Scripts" AND EDIT THE COPY. **
]]--

--set the title of the HUD
local title = "Recent Private Messages:"
local showTitle = true

--set the number of messages to show
local messageCount = 4

--set the maximum message length (including sender name)
local messageLength = 60

--set the top left x and y coordinates of the HUD
local location =
{
x = 25,
y = 5
}

--set the colors for the HUD
local colors =
{
message = {95, 247, 247},
title = {225, 225, 225},
}

-------------------------------------------------------------------------------------------------
-------------------------------------------------------------------------------------------------
-------------------------------------------------------------------------------------------------

--this creates the title HUD
if (showTitle == true) then
local HUDTitle = HUD.CreateTextDisplay(location.x, location.y, title, unpack(colors.title))
location.y = location.y + 20 -- just offsets the y axis to below the header
end

--loop from 1 until messageCount, creating HUD objects for each
local HUDData = {}
for index = 1, messageCount do
local data = {}

-- no message yet
data.message = ""

--create the label HUD
data.label = HUD.CreateTextDisplay(
location.x,
-- +12 pixels to the y axis for every item
location.y + ((index - 1) * 12),
data.message,
-- text color
unpack(colors.message)
)

--store the data to use later
HUDData[index] = data
end

--this is a proxy event which gets invoked when a private message is received
PrivateMessageProxy.OnReceive("pmProxy", function (proxy, speaker, level, text)
--create the message from the data
local currentTime = os.date("*t")
local message = currentTime.hour .. ":" .. currentTime.min .. " | " .. speaker .. " [" .. level .. "]: " .. text

--if the message is too long, this will shorten it
if (string.len(message) > messageLength) then
message = string.sub(message, 0, messageLength - 3) .. "..."
end

--this will move each message "up" (from index 4 to 3, 3 to 2, 2 to 1, etc)
for index = 1, messageCount-1 do
HUDData[index].message = HUDData[index+1].message
HUDData[index].label:SetText(HUDData[index].message)
end

--this will put the new message in the last index
HUDData[messageCount].message = message
HUDData[messageCount].label:SetText(message)
end)

The line


local message = currentTime.hour .. ":" .. currentTime.min .. " | " .. speaker .. " [" .. level .. "]: " .. text

Remove the | and one of the spaces next to it, send yourself a PM, and observe crash.

Edit: Not crashing for me. Did for Luls. Not crashing for him now. What the very fuck is happening?!

[Luls]

We were unable to reproduce the crash :S

Closed.