-
Notifications
You must be signed in to change notification settings - Fork 80
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
When there is no text or no command after \enumerate, pagebreak error occur. #323
Comments
my SATySFi's version:
|
The assertion causing this error was introduced at the same time as |
As @leque pointed out, --- a/src/backend/pageBreak.ml
+++ b/src/backend/pageBreak.ml
@@ -105,7 +105,8 @@ let chop_single_column (pbinfo : page_break_info) (area_height : length) (pbvbls
| (PBVertSkip(_), _) :: pbvbtail -> omit_redundant_clear pbvbtail
| (PBVertLine(_), _) :: _
| (PBVertFrame(_), _) :: _ -> None
- | (PBHookPageBreak(_), _) :: _ -> assert false
+ | (PBHookPageBreak(_), _) :: _ -> None
| [] -> Some([]) (* -- the original `pbvblst` consists only of spaces -- *)
in Change like above eliminates the error message and it looks there is no problem in output pdf. But, I'm not confident that this is the right way to solve this issue. Also, I did somewhat simplified reproduction of this problem.
The assertion was about In the original situation, (edit) Code like below also reproduces the problem.
is sufficient.
|
Example
Input
Output
@ R_O_R_I_J_O also found this error: https://twitter.com/R_O_R_I_J_O/status/1505121840565735424?s=20&t=gukKQDDsyNMagWU_WiHiWQ
I could avoid this error by following ways:
@ R_O_R_I_J_O checked no error using \listing in place of \enumerate: https://twitter.com/R_O_R_I_J_O/status/1505039343991533571?s=20&t=gukKQDDsyNMagWU_WiHiWQ.
This error occured when \enumerate is placed in +p, +definition, +lemma, +theorem, +corollary commands. However, such document can be compiled in +proof command. For example,
Output
The text was updated successfully, but these errors were encountered: