[wpe-webkit] Unable to build stable (semi-colon gotcha?)

[Andres Gomez]

Hi,

quickly answering:

U16_NEXT is a preprocessor macro. In other words, it is replaced by the
actual code in that stage of the compilation.

If you check the definition of this macro at:
https://git.webkit.org/?p=WebKit.git;a=blob;f=Source/WTF/icu/unicode/utf16.h;h=bdd88a8b9c4f00ea0df272ac02f6967ef61089b6;hb=HEAD#l264

You will realize that it is replaced by a block of code (starting and
closed with brackets).

In other words, although it would be good to have the semicolon there,
it is not really needed and it should not lead to a build failure,
AFAIS.

So, are you sure you don't have a different problem building the code?

Br.

On Mon, 2020-01-13 at 14:26 +0100, Andrea Giammarchi wrote:
> Happy new year everyone!
> 
> Quick question: while I see that current code has semi colons at the right place, it's not the first file I need to mokey patch from the stable 2.26.2 archive.
> 
> As example, the Document.cpp file has 3 lines where U16_NEXT... misses the semi-colon at the end, resulting in build failures.
> 
> My question is:
> how is it possible that a stable release doesn't build out of the box?
> do I understand correctly source files from WebKit or WPE fork are not linted?
> Specially the latter point, would be quite surprising to me ... as I'm automating ways to build WPEWebKit for various architectures, and dealing with files that cannot be used and break the build down the process is not really expected/convenient.
> 
> Thanks for any sort of clarification.
> 
> Best Regards
>  _______________________________________________
> Do not post admin requests to the list. They will be ignored.
> webkit-wpe mailing list      (webkit-wpe at lists.webkit.org)
> Help/Unsubscribe/Update your Subscription:
> https://lists.webkit.org/mailman//listinfo/webkit-wpe
-- 
Br,

Tanty
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 833 bytes
Desc: This is a digitally signed message part
URL: <http://lists.webkit.org/pipermail/webkit-wpe/attachments/20200113/ad7d1c7c/attachment.bin>