[tarantool-patches] Re: [PATCH 2/2] sql: remove GLOB from Tarantool
Alexander Turenko
alexander.turenko at tarantool.org
Fri Aug 17 14:34:20 MSK 2018
On Fri, Aug 17, 2018 at 02:26:19PM +0300, Alexander Turenko wrote:
> Hi!
>
> I have one note.
>
> WBR, Alexander Turenko.
>
> On Fri, Aug 17, 2018 at 11:25:14AM +0300, Alex Khatskevich wrote:
> >
> >
> > > if (sqlite3Utf8CharLen((char *)zEsc, -1) != 1) {
> > > sqlite3_result_error(context,
> > > - "ESCAPE expression must be a single character",
> > > + "ESCAPE expression must be a"
> > > + " single character",
> > Do not split error messages at the middle of a sentence. It makes errors
> > ungreppable.
> > Make it <80 somehow different.
> >
>
> It was my thought that such rule should be enforced. But I asks Vladimir
> D. and he says that kernel developers don't care about that and show me
> examples.
>
> Anyway, how you would propose to avoid the string splitting here?
>
> > > - sqlite3_result_error(context, "LIKE or GLOB pattern can only"
> > > - " contain UTF-8 characters", -1);
> > > + sqlite3_result_error(context, "LIKE pattern can only contain"
> > > + " UTF-8 characters", -1);
> > Do not split error messages at the middle of a sentence. Make it <80 somehow
> > different.
>
> Same here.
>
Sorry, missed the following discussion.
If you think we should change our code style, please review it with
mantainers first. But as I said above we already have this point
discussed.
WBR, Alexander Turenko.
More information about the Tarantool-patches
mailing list