Re: [PATCH] gropdf: Fix papersize control command

2023-08-09 Thread G. Branden Robinson
Hi Deri, At 2023-08-05T16:09:41+0100, Deri wrote: > On Saturday, 5 August 2023 15:55:20 BST G. Branden Robinson wrote: > > Deri, I'm happy to apply a corrective documentation change to > > gropdf(1). Is that cool with you? Let me know. > > > > I think it'd be a good idea to explicitly note the

Re: [PATCH] gropdf: Fix papersize control command

2023-08-05 Thread Alexis
Hello Branden and Deri, for a brief moment I was considering whether an additional control command could help create consistency, but after giving it some thought I believe it would only add to the confusion. Hence documentation calling out the inconsistency explicitly seems the sane path forward.

Re: [PATCH] gropdf: Fix papersize control command

2023-08-05 Thread Deri
On Saturday, 5 August 2023 15:55:20 BST G. Branden Robinson wrote: > Deri, I'm happy to apply a corrective documentation change to gropdf(1). > Is that cool with you? Let me know. > > I think it'd be a good idea to explicitly note the order reversal of the > dimensions (probably in grodvi(1) as w

Re: [PATCH] gropdf: Fix papersize control command

2023-08-05 Thread G. Branden Robinson
Hi Deri & Alexis, At 2023-08-05T16:36:31+0200, Alexis wrote: > Thank you for the helpful context, Deri, much appreciated. I checked > the sources thoroughly, in order to have a good reasoning, but missed > to read the grodvi sources :/ I would prefer for groff to have internal consistency than co

Re: [PATCH] gropdf: Fix papersize control command

2023-08-05 Thread Deri
On Saturday, 5 August 2023 15:36:31 BST Alexis wrote: > Thank you for the helpful context, Deri, much appreciated. I checked > the sources thoroughly, in order to have a good reasoning, but missed > to read the grodvi sources :/ > > Following the same order, i.e. width,height, to allow the same so

Re: [PATCH] gropdf: Fix papersize control command

2023-08-05 Thread Alexis
Thank you for the helpful context, Deri, much appreciated. I checked the sources thoroughly, in order to have a good reasoning, but missed to read the grodvi sources :/ Following the same order, i.e. width,height, to allow the same source to run on either dvi or pdf makes a lot of sense; too bad t

Re: [PATCH] gropdf: Fix papersize control command

2023-08-05 Thread Deri
On Saturday, 5 August 2023 11:12:07 BST Alexis wrote: > In brief: The \X'papersize=paper-format' control command seems to > read the paper-format parameter as width,length as opposed to the > gropdf -p option and the referenced documentation (groff_font(5)) > which specifies length,width for the pa

[PATCH] gropdf: Fix papersize control command

2023-08-05 Thread Alexis
Hi folks, I may have stumbled upon a bug related to gropdf's papersize control command. Please note that my understanding of how things are supposed to work may very well be too limited and lead me to false assumptions. In brief: The \X'papersize=paper-format' control command seems to read the pa