fmt in debugmouse:draw3.

This is literally the reason I implemented fmt in the first place
use fmt in a commented-out debug logging block
2023-01-01 13:31:34 -08:00 · 2023-01-01 13:29:52 -08:00 · 2023-01-01 13:27:32 -08:00 · 2023-01-01 13:18:31 -08:00 · 2023-01-01 12:53:16 -08:00 · 2023-01-01 12:50:35 -08:00
1 changed files with 9 additions and 35 deletions
--- a/chameleonic.p8
+++ b/chameleonic.p8
@ -1777,41 +1777,20 @@ function rot13(s)
 end
 return chr(unpack(sord))
 end
-
 --hint file format:
-- each row is one hint.
-- 4 or 5 columns
-- separated with a
-- grave (`) character
-- [1] room# 
-- [2] x coord
-- [3] y coord
-- [4] message line 1 (rot13)
-- [5] message llne 2 (rot13)
-- row 5 can be omitted
-- for a 1-line hint
--
-- multiple hints for the same
-- room are revealed in order
+-- each row is one hint. 4 columns per row
+-- separated with a grave (`) character
+-- room# ` X (screen px) ` Y (screen px) ` message
+-- message is ROT13d; works for a-zA-Z

 hints = {}
 add(real_modules,hints)

 function hints:init()
- local h = gsv[[0`42`57`🅾️ yVPX`  ■
-0`42`73`❎, ❎ cHYY
-1`35`34`⁘ sVYY
-1`99`82`■ cHYY
-1`42`98`⁘`VTABER
-2`75`65`i <`⁘
-2`104`73` ■`cHYY
-2`27`42`⁘
-3`51`106`■ cHYY
-3`27`81`⁘ HAOYBPX ZR
-3`91`33`■ FGNAQ`  URER]]
+ local h = gsv[[0`64`64`zYRZ
+0`32`32`fCYHOC]]
 for rec in all(h) do
  rec[4]=rot13(rec[4])
-  if(rec[5]) rec[5]=rot13(rec[5])
  local lh = self[rec[1]]
  if lh then
   add(lh,rec)
@ -1823,11 +1802,6 @@ function hints:init()
 menuitem(2,"hide hints",function() level.hintlevel=0 end)
 end

-function shdprint(txt,x,y,c)
- print(txt,x-1,y+1,1)
- print(txt,x,y,c)
-end
-
 hintflicker=split"7,10,9,8,8,9,10,7"
 function hints:draw2()
 pal()
@ -1835,9 +1809,9 @@ function hints:draw2()

 for i,h in ipairs(self[level.ix]) do
  if (i > level.hintlevel) return
-  local _,x,y,txt,txt2=unpack(h)
-  shdprint(txt,x,y,c)
-  if (txt2) shdprint(txt2,x,y+8,c)
+  local _,x,y,txt=unpack(h)
+  print(txt,x-1,y+1,1)
+  print(txt,x,y,c)
 end
 end
Author	SHA1	Message	Date
Kistaro Windrider	0d3b0e4dac	fmt in debugmouse:draw3. This is literally the reason I implemented fmt in the first place	2023-01-01 13:31:34 -08:00
Kistaro Windrider	ec98554c78	use fmt in a commented-out debug logging block	2023-01-01 13:29:52 -08:00
Kistaro Windrider	4363f2ed43	golf recollide_reanchor Uses fmt in recollide_reanchor. This has a performance impact in code that is already slow so this might need to be reverted; I think that should be part of a comprehensive optimization pass, however, and making it worse for now as part of a general code cleanup is probably better.	2023-01-01 13:27:32 -08:00
Kistaro Windrider	b7ce1fe85e	tostring golf this could be golfed further by rewriting the str..= line in terms of fmt, but the performance impact of inserting the extra `fmt` calls and parsing into the tostring recursion seems likely to be a problem.	2023-01-01 13:18:31 -08:00
Kistaro Windrider	179ee35985	Use fmt on title screen.	2023-01-01 12:53:16 -08:00
Kistaro Windrider	367f03be4a	fix section before first "%" oops, need to special-case the first part of the split list. amusingly, I don't need to special-case zero-length format strings because that will skip the entire loop and output "", which seems correct. (I am also not special-casing zero-length segments because `s[1] == nil` will go into the error handler and that seems fine.	2023-01-01 12:50:35 -08:00
Kistaro Windrider	2ce259304d	fix double-consume, add %! I discovered your tostring function for debugging, might as well make it reachable from fmt. I also realized I forgot to convert to using "p" when I introduced it so I fixed that	2023-01-01 12:42:37 -08:00
Kistaro Windrider	e0ebb035e1	fix sprintf and rename to fmt the "split on %" strategy makes parsing "%%" complicated, so the "actually literally %" placeholder is now "%~" rather than "%%". since thsi resembles a real sprintf even less now I renamed it to "fmt". also actually closes the `if m == "~"` block. (which was `if m == "%"` before this patch)	2023-01-01 12:37:11 -08:00
Kistaro Windrider	b0bca27239	implement fake sprintf this spends extra tokens to handle "invalid format character" to catch using a % where %% is intended. this is designed to grow with further format chars if needed; I have ideas for not-exactly-POSIX %d, %x, %h, %!, %#, %c, and possibly others but there is absolutely no reason to spend tokens on these things until we need them. (that said, existing debugging output might benefit from some of these other formats, but that debug code is commented out, so maybe nevert.)	2023-01-01 12:33:16 -08:00
Kistaro Windrider	aefdfeb99c	Better debug mouse Debug mouse is now its own module, so it can be separated from the hint system, since it is useful for more than just positioning hints. It now has the following enhancements: 1. Clock-based table cyclng now has a helper function (cycle) 2. Debug mouse color cycling is distinct from hint color cycling, so debug position readout remains legible 3. Debug position readout now stays on screen even when the cursor is near or past the edges 4. Debug cursor cycles between a mouse sprite specifically marking the exact pixel that is being sampled, an "X" for text character sizing, and a "□" for positioning the centered 3x3 characters often used as hint target markers 5. Map cell coordinates (in square brackets) are displayed in addition to pixel coordnates (in parentheses) Sprite 50 is now the mouse cursor. Color 15 is color cycling for debug readouts. Debug mouse features can be disabled by commenting out `add(real_modules, debugmouse)`. I've done a little bit of golfing but this is stiill a token expense. I'm going to write a crappy sprintf function to save tokens everywhere we're assembling strings from their component parts.	2023-01-01 12:06:22 -08:00