akkartik/carousel.love
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

avoid some unnecessary allocations

Browse Source 
To find the first word we currently strip _all_ Lua comments. This is
really wasteful for large definitions. At least avoid all these
allocations if the first word is not a comment.
This commit is contained in:
Kartik K. Agaram 2023-07-24 00:44:34 -07:00
parent 5cac6d8584
commit f23ed15fd9
1 changed files with 8 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

9
live.lua
View File

@ -230,11 +230,18 @@ function live.run(buf)
end
function live.get_cmd_from_buffer(buf)
-- return the first word
return buf:match('^%s*(%S+)')
end
function live.get_definition_name_from_buffer(buf)
return buf:gsub('%-%-[^\n]*', ''):match('^%s*(%S+)')
-- Return the first word, after skipping Lua comments.
if not buf:match('^%s*%-%-') then
return buf:match('^%s*(%S+)') -- avoid unnecessary allocations
end
-- wastefully strip out comments everywhere, not just at the start
buf = buf:gsub('%-%-[^\n]*', '') -- line comments
return buf:match('^%s*(%S+)')
end
function live.get_binding(name)