🚀 go-pugleaf

Maciek Godek pisze:
> W dniu środa, 14 sierpnia 2019 19:24:46 UTC+2 użytkownik PawelS 
> 
>> Prawda. Nie wiem czy do tego zmierzasz, prawdopodobnie tak, że -Wall
>> to wcale nie włącza wszystkich dostępnych dla gcc opcji -W***
> 
> Tak, trzeba przyznać, że już sama nazwa jest myląca, no bo na moją znajomość języka angielskiego "all" oznacza wszystkie.
> Ale to jeszcze idzie przełknąć, zwłaszcza jak się wie o fladze -Wextra, która włącza dodatkowe warningi, których -Wall nie włącza.
Tak jak napisałem, jest to prawdopodobnie celowa nie konsekwencja:
int main() {
	char buffdata[256];
	char buffcopy[256];
	/*unsigned*/ char *sp1, *sp2;
	size_t ss;
	
	
	strncpy(buffdata, "test\"1234\"test", sizeof(buffdata));
	
	if ((sp1=strchr(buffdata,0x22))!=NULL && (sp2=strchr(sp1+1,0x22))!=NULL) {
		ss=sp2-sp1;
		if (sp2-sp1<sizeof(buffcopy)) strncpy(buffcopy, sp1+1, sp2-sp1-1);
		else                          strncpy(buffcopy, sp1+1, sizeof(buffcopy));
		printf("buffcopy=|%s|\n", buffcopy);
	}
	
	return 0;
}

test.c:16: warning: comparison between signed and unsigned
test.c:16: warning: passing argument 3 of '__builtin_strncpy' as unsigned due 
to prototype

Włączenie -Wconversion oznaczałoby często w powyższym przypadku
stosowanie rzutowania (size_t), ewentualnie stosowanie dodatkowej
zmiennej dla obliczania długości ciągu znaków. Co ciekawe
kompilator nie zgłasza tutaj ostrzeżenia konwersji:
ss=sp2-sp1;
a sp1, sp2 (char *) są ze znakiem,
natomiast ss (size_t) jest bez znaku.

Zgodnie z wcześniejszym wynikiem kompilacji:
uint32_t f(int32_t x) {
     return x*x;
}

x.c:3:13: warning: conversion to ‘uint32_t {aka unsigned int}’ from ‘int32_t 
{aka int}’ may change the sign of the result [-Wsign-conversion]
      return x*x;