currency: Change Value::html to demand a ShowUnocculted

This finds places we need to hide the quantity.  One (new_value) was
wrong, and is now fixed.

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

currency: Change "new_qty" to "new_value"

Spotted a bug, which leaked value during occultation.  Putting new_qty
inside Value makes it easier to arrange to always call call .html on
it.

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

currency: Rename to show_to_player

This is how it's going to be when we occult.

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

currency: Move slight, to prepare for occultation

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

currency: Restructure slightly to make next log changes easier

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

currency: Offer occulted version

But we don't do merging within occultations yet.

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

currency: Promote image conversion

Occult will want it earlier.

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

currency: Break out describe()

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

currency: Break out value.html()

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

currency: Break out render

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

currency: Support adjusting size of the unit text, separately

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

dice: Arrange to use text_options.start_element

What a palaver!

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

dice: Insist that x and y are as our Rust would calculate them

Preparatory for using TextOptions::start_element, which expects
to centre text at the nominal position.

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

Break out SVG_FONT_Y_ADJUST_OF_FONT_SIZE into a separate file.

The dice template extractor is going to want this.

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

dice: Don't include p in process_node.

The text noce is going to want to do something odd.

No functional change.

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

TextOptions: break out y_adjust

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

dice: Note a todo

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

Break out TextOptions::start_element (from currency)

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

Use TextOptions in currency, so it now supports setting text size

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

Move text size into TextOptions, so dice now support setting text size

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

childio: Work around Linux kernel race bug

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

docs: Fix a minor case oddity

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

Break out TextOptions and TextOptionsSpec

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

currency: support label text recolouring

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

Break out ColourSpecExt::resolve

Hrm, maybe this should have been load() ?  This will do for now.

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

demo.game.toml: Remove non-hnoured min_unit setting

We decided to leave this to the players.

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

Html: Fix Debug truncation to not randomly panic

Ow!

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

spec loop detecton: Write down the rule

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

spec loop detecton: Detect for magic too.

Introduce PieceLoadArgs::recursing, and use it in the one current call
site.

Make the error more general.  So now we can call it for magic too.

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

clippy: Misc minor improvements

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

clippy: Remove a redundant pair of parens

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

cli bundles: autozip: Use Deflate

This is considerably faster.

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

progress reporting: Nest: Provide and use more desc plumbing

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

cli bundle progress: Say we're hashing, when we are.

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

cli bundle progress: Report for files in the zip we're building

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

cli bundle progress: Collect local zipfile members in a vec

This will let us easily count them, which we want to do for progress
reporting.

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

progress: Nest: Fix to actually be progressive

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

cli bundle progress: clear before thinking about bundles

Otherwise the output is a bit messed up.

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

progress: Nest: Issue a report at outer phase transitions

There's not much to say here, but without this, if the inner phase
does nothing, the user sees nothing.  Better to do something here.

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

otter cli: Use new Nest progress during bundle uploading

This lets us show progress of zipfile preparation.

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

progress: Replace Nest

This version doesn't need everyone to have counts.  In particular, you
can have woolly phases.  The otterlib bundle processing will want
this.

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

progress: Rename NestEqual from Nest

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

progress: Allow ProgressInfo's Count to contain only a proportion

We're going to introduce a thing that will use this.

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

bundle auto-zip: works, but needs some progress reporting

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

bundle auto-zip: wip

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

dice: Set label colour of red dice.  Hooray!

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

dice: Allow setting label colour

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

dice: Use new templating arrangements to do away with d8 group

Now we can have different dice all ni the smme group.

They all have to have the same size, so we adjust the d8 svg
accordingly.  (The d6 svg has to be the way it is because it's
extracted from the die cooldown etc. template.)

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

bundles: Fix a bug which could cause IE on bundle load

We need to make an entry entry in the bundles notes array even if the
hash is missing.  Or we might try to reuse the bundle number, if there
was a failed bundle upload followed by a server restart.

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

shapelib bundle tests: Do not filter out "filenames" after ^:

These are actually extra field definitions.  If there are these, our
parser is too stupid.  We're just filtering out the source filenames
to know that otter isn't reading them.

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

shapelib subst: Reorder other settings

This lets the explicit substs from various places override the builtin
image and colour.

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

shapelib subst: Reorder overrides

This is a bit confusing.  Early settings take precedence since they
get substituted away!

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

shapelib, subst: Allow colours to provide magic substs

Eg, for text colour.  The dice want this.

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

shapelib, subst: Allow groups to provide magic substs

Otherwise they might have to recapitulate the template.

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

shapelib subst: Prepare for substituting more into magic

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

docs: Document die numeric labels

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

run-inkscape-extension: Work around a weird inkscape behaviour

Without this, generic recolour instructions in shape libraries don't
work right if some of the recolourings don't apply to some of the
image files in the group.  Which is about to be the case.

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

subst: Use ${...} format internally for most things

This saves allocating in the usual case and is also clearer.

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

subst tests: Add test cases for converting needles

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

subst: Have substitutor handle any needle syntax

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

subst: Have subst_general return the resolved needle

This corrects the error reporting to show what we actually looked for.

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

subst test: Add many dbg calls

It turns out that I can make these fail and having it debug is useful.

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

dice: Change the inner edges in the blue d8

This looks better.  We will have to recolour them for the red die.

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

subst: Substitute x* variables into magic template

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

dice: red wip

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

dice: d8 wip

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

subst: Allow non-'static keys

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

magic: Use new ${image} syntax

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

subst test: Test format 2 more thoroughly

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

subst tests: Disable under miri

We're going to add dbg! calls, which break.

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

subst test: Test format 2

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

subst test: Test mf1 Filename

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

subst test: Reintroduce and fix up for new API

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

subst: Make some things into methods

Wanted to do this for subst and substn but this terminally confuses
the borrow checker.

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

subst: Always substitute dollars in mformat 2 text

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

subst: Provide Dollars::enabled

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

subst: Hoist dollars into c_* aliases

So now colour_subst_1 knows if it's text or not.
No functional change.

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

subst: Demote PerhapsSubst

Reverts
  subst: Promote PerhapsSubst

We shouldn't do this for anything but colour specs.

And, really, we ought to do it only for the item name and not for the
other things, or the ${$} de-escaping is very confusing.

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

subst: Track whether we're working in a filename

Ie, we are going to do this in a principled way, depending on what
kind of thing we are trying to generate, rather than just whether we
saw something other than "_c".

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

subst: Provide and use internal_err()

We're going to have more things that might be wrong at runtime...

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

subst: Break out Substituting::err

We're going to want to reuse this.

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

Make InternalLogicError be Clone, with an Arc

shapelib wants this for SubstErrors which ought to be Clone.

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

subst: Implement format 2 and convert all libraries

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

subst: Have internal functions take borrows

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

subst: Plumb mformat into Substituting, and change it to be Cow

This will allow us to do know whether to use the new format.

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

subst: Temporarily comment out the test

There is so much reorg here.  I don't want to keep editing this.

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

subst: Track in type whether we're substituting

This stops us from failing to do a final cleanup step (which is going
to be replacing `${$}` when we have mformat 2 style.  It also stop us
restarting substitution unexpectedly, and marks places where text
enters the substitution machinery.

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

subst: Promote PerhapsSubst

I thought we wanted this for sorts.  I find we don't, but I don't want
to drop this change and deal with the conflicts.  I will revert it
later.  Much later.

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

subst: Make PerhapsSubst::finish fallible

It is going to throw SubstError.

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

subst: Introduce PerhapsSubst which will let us have subst chaining

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

dice: d8 wip

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

dice: d8 wip

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

dice: d8 wip

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

otterlib: Show large objects a bit bigger

This algorithm is a bit paants.

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

otterlib: Show three views only for first two faces

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

otterlib: Fix column handling in preview

This was very confused.

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

dice: Get size right in preview with a special entrypoint

This is the code from
  dice: Get size right in preview by setting bbox, breaking other things
but in a new entrypoint used only for preview.  The comment explains why.

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

Revert "dice: Get size right in preview by setting bbox, breaking other things"

This reverts commit 1df93eed42dc37563a7bf5df825f800cb871396d.

dice: Get size right in preview by setting bbox, breaking other things

This is wrong.

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

dice: Unwrap surround_path

We're making changes here and it's confusing.  Make this more
prominent.

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>