Skip to content
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

Updated Code; Duplex, Simplex #180

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

Conversation

QORTEC
Copy link
Contributor

@QORTEC QORTEC commented Mar 14, 2023

Duplex, Simplex

The following code explicitly sets the duplex/simplex in PCL.

if (page_params_.duplex && page_params_.tumble) {
  fputs("\033&l2S", out_);
} else if (page_params_.duplex) {
  fputs("\033&l1S", out_);
} else {
  fputs("\033&l0S", out_);
}

\033&l2S: This sets the printer to duplex printing with tumble mode.
\033&l1S: This sets the printer to duplex printing without tumble mode.
\033&l0S: This sets the printer to simplex (single-sided) printing.

#169 explicitly sets the printer in simplex mode (\033&l0S).
This resolves the issue of printers continuing to duplex when disabled.

QORTEC#2 adds duplex no tumble printing (\033&l1S).
This change also adds logic to print with and without tumble.

Compiling

QORTEC#3 fixes building on Fedora 38;
error: 'uint8_t' was not declared in this scope
Resolves , so uint8_t undeclared issue by explicitly including cstdint in test/tempfile.h.

tomjnixon and others added 3 commits March 14, 2023 03:32
this seems to be required for the HL-L2350DW, and matches the behaviour
of the official driver

without this, if duplex is disabled then the printer still duplexes, but
every other page is garbled
Building brlaser under Fedora 38 gives the following error in
test/tempfile.h:
error: 'uint8_t' was not declared in this scope

This commit fixes it by explicitly including `cstdin` in the file, so
`uint8_t` is correctly defined there.
@knkrth
Copy link

knkrth commented Oct 8, 2024

How to copy 2 side -> 2 side layout. On the official firmware there is only 1 side -> 2 side layout option available.

Corrected the Duplex & Tumble Check:
 - swapped the PLC values for duplex and tumble.

Thanks @wmlapierre
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants