Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>

../cvt/cvt.c:34:1: warning: no previous prototype for ‘cvt_is_standard’ [-Wmissing-prototypes]
   34 | cvt_is_standard(int hdisplay, int vdisplay, float vrefresh, bool reduced, bool verbose)
      | ^~~~~~~~~~~~~~~

Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>

Olivier Fourdan authored 2 years ago and Olivier Fourdan committed 8 months ago

Update the CI to add the template Security/SAST.gitlab-ci.yml as most of
the other xorg projects do.

Signed-off-by: Olivier Fourdan <ofourdan@redhat.com>
Part-of: <!12>

We're not getting paid enough to use that many dollars.

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>
Part-of: <!15>

nyanpasu64 authored 1 year ago and Olivier Fourdan committed 1 year ago

Signed-off-by: nyanpasu64 <nyanpasu64@tuta.io>

nyanpasu64 authored 1 year ago and Olivier Fourdan committed 1 year ago

Previously the code erroneously used CVT_MIN_V_PORCH=3 instead of the
specified CVT_MIN_V_BPORCH=6, when calculating the minimum size of the
back porch in scanlines. As a result, when generating 320x240@60, the
vtotal would be be incorrectly set to 252 instead of 253.

This commit fixes the typo.

Signed-off-by: nyanpasu64 <nyanpasu64@tuta.io>
Closes: #7


Reviewed-by: Olivier Fourdan <ofourdan@redhat.com>

The CVT v1.2 specification says, in section 3.4.1 "Standard CRT-based
Timing":

> The Horizontal Sync Pulse duration will in all cases be set as closely
> as possible to 8% of the Total Horizontal time, rounded down to the
> nearest cell width.

The previous code achieved this by first setting mode_info->hsync_end to
a multiple of the cell width, then calculating mode_info->hsync_start
and rounding its position up to a cell width. However, if
mode_info->hsync_start was already a multiple of 8 (for example on
640x480@60), the rounding code would erroneously add 8 to it, shortening
the hsync duration by 8 pixels.

This commit fixes the bug by calculating and rounding the sync width
separately, closer to the specification's wording.

Signed-off-by: nyanpasu64 <nyanpasu64@tuta.io>
Closes: #6


Reviewed-by: Olivier Fourdan <ofourdan@redhat.com>

Signed-off-by: Olivier Fourdan <ofourdan@redhat.com>

Signed-off-by: Jeremy Huddleston Sequoia <jeremyhu@apple.com>

Signed-off-by: Dima Krasner <dima@dimakrasner.com>

Signed-off-by: Olivier Fourdan <ofourdan@redhat.com>

Simon Ser authored 3 years ago and Olivier Fourdan committed 3 years ago

It was set to 1.0 instead of 0.1.0.

Signed-off-by: Simon Ser <contact@emersion.fr>

This allows using libxcvt as a subproject.

Signed-off-by: Simon Ser <contact@emersion.fr>

Signed-off-by: Xaver Hugl <xaver.hugl@gmail.com>
Reviewed-by: Olivier Fourdan <ofourdan@redhat.com>

Olivier Fourdan authored 3 years ago and Olivier Fourdan committed 3 years ago

This is a very early release likely to change in the future, make sure
to use library versioning.

Signed-off-by: Olivier Fourdan <ofourdan@redhat.com>

A very simple CI for now, just make sure it builds.

Largely inspired by the implementation in libinput by Peter Hutterer
<peter.hutterer@who-t.net>.

Signed-off-by: Olivier Fourdan <ofourdan@redhat.com>
Reviewed-by: Benjamin Tissoires <benjamin.tissoires@gmail.com>
Reviewed-by: Peter Hutterer <peter.hutterer@who-t.net>

Fix man page substitutions

See merge request !2

Fills in version & section info in cvt.1 and fixes meson warning:
man/meson.build:4: WARNING: The variable(s) 'appmansuffix', 'filemansuffix', 'vendorversion' in the input file 'man/cvt.man' are not present in the given configuration data.

Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>

readme: Project has moved

See merge request !1

Project has moved under the xorg/lib namespace, update the instructions
accordingly.

Signed-off-by: Olivier Fourdan <ofourdan@redhat.com>