-
Notifications
You must be signed in to change notification settings - Fork 178
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
l3skip: view the dimension of LaTeX2e macro #417
Labels
enhancement
New feature or request
Comments
You are using |
You are using |\dim_set:Nn| on something that is a skip: I'm not sure
that anything other than 'trouble' can arise ...
well it wouldn't need to and it is kind of reasonable to expect that
\dim_set:Nn would set the main part of a skip and the plus/minus to zero
all that is missing is a \scan_stop: in the definition, isn't it? that
would guard against it like it guards against it in \skip_set:Nn.
So even if that is a bit of a questionable use the outcome should either
be an error or sensible not kind of random.
|
@FrankMittelbach There is a |
Am 24.11.17 um 15:45 schrieb Joseph Wright:
@FrankMittelbach <https://github.com/frankmittelbach> There is a
|\__dim_eval_end:| = |\scan_stop:| there: I guess you mean you need two
to catch this situation?
|
You are using \dim_set:Nn on something that is a skip:
Whichever operand is which, both should do something sensible (by reasonably normal coercion).
Certainly not a cause for dismissal as ‘trouble’. More an opportunity for some politesse and a fix.
|
blefloch
pushed a commit
that referenced
this issue
Nov 29, 2017
Some past commit only added \scan_stop: to \dim_set:Nn, but there is also \dim_add:Nn, \dim_set_eq:NN, \dim_zero:N and friends.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Try the following code:
Only the second
\dim_show:N
gives the right value:If try this
The result of
\dim_show:N
is also correct.Of course, when using the "appropriate way" as
\skip_set:Nn \rightskip { \c__foo_test_tl }
instead, both results is OK then.LaTeX3 version: 2017/11/14.
The text was updated successfully, but these errors were encountered: