[sane-devel] [janitorial] Feature freeze for 1.0.29 and call for translation updates

Yuri Chornoivan yurchor at ukr.net
Mon Jan 13 20:19:17 GMT 2020


понеділок, 13 січня 2020 р. 21:53:43 EET Ulf Zibis написано:
> Hi,
> 
> Am 13.01.20 um 20:26 schrieb Ralph Little:
> > On Mon, Jan 13, 2020, 11:17 Ulf Zibis, <Ulf.Zibis at gmx.de
> > 
> > <mailto:Ulf.Zibis at gmx.de>> wrote:
> >     Am 13.01.20 um 19:58 schrieb Ralph Little:
> >>     On Mon, Jan 13, 2020 at 9:58 AM Ulf Zibis <Ulf.Zibis at gmx.de
> >>     
> >>     <mailto:Ulf.Zibis at gmx.de>> wrote:
> >>         Here came some surprise. With diff, I could see, that many
> >>         lines were
> >>         changed, even there was no semantic change. This was caused
> >>         by poedit
> >>         from automatic line break. It breaks lines after 83 chars,
> >>         regardless if
> >>         I e.g. set it to 70 (with "keep format of existing files"),
> >>         the default
> >>         was 79.
> >>         So I like to suggest, to first "reformat" the de.po file for
> >>         a stable
> >>         format, make a merge request and after do the semantic
> >>         changes, so
> >>         reviewers would have less work.
> >>     
> >>     I tend to agree!
> >>     I prefer to formatting changes separated from functional updates.
> >     
> >     Great!
> >     I personally would prefer NO line breaks, as comparing the diffs
> >     after
> >     would be much easier (otherwise a little change at the beginning of a
> >     long text changes all line breaks for several following lines, which
> >     makes it error-prone to observe changes at the end of the long text)
> >     
> >     If you disagree and want automatic line breaks, which width do you
> >     want?
> >     
> >     -Ulf
> > 
> > I personally don't have a view.
> > However I see that the default setting in poedit is 79 and that seems
> > reasonable to me.  In the version I have has a setting of "Preserve
> > formatting of existing files" so I don't know why it would be making
> > other changes. :(
> 
> Maybe this setting scans for the longest line in the input file an takes
> its width instead the determined 79.
> 
> Is there any opposition from someone against disabling the automatic
> line break, as today's screens are huge and modern edit and visual diff
> tools are anyway capable to visually wrap lines according the window
> width. This would dramatically serve visual diff tools to highlight tiny
> differences in long logical strings.
> 
> -Ulf

Hi,

Olaf proposed to use 75 chars.

https://gitlab.com/sane-project/backends/merge_requests/305

Personally, I have configured my Lokalize project to use this width.

Best regards,
Yuri






More information about the sane-devel mailing list