[sldev] CMake preview available - please try it out!

[Robin Cornelius]

Bryan O'Sullivan wrote:

>> What would be nice is a way of modifying the gcc flags so that -Werror
>> can be turned off if desired, not sure if this is a good idea or not?
> 
> I'd rather fix the causes of the errors.  Quite often, gcc's warnings
> are worth paying attention to.

Agreed gcc's warnings are always worth looking at, but In this case they
are  string conversions warnings.

 warning: deprecated conversion from string constant to ‘char*’

and

 deprecated conversion from string constant to ‘gchar*’


Yes fixing the warnings would be great, but its a massive patching
operation to remove or cast all the constant strings. Due to the number
of files effected I suspect this will have to be an internal operation
as any patch will be out of date almost instantly. In  95% of cases its
just adding a (cont *) cast to constant strings, or making better use of
std::string. I believe someone at the labs said they were looking at
this but it became a lower priority than other things happening. There
are some cases where a simple (const *) does not solve the issue but
probably a wholesale conversion to std::string solves these anyway.

The solution in this case may be (if gcc allows) to suppress the
specific warning? We know why its caused, older gcc's do not even show
it. Perhaps we can just mask it, ideally i would like it a a non-fatal
warning with other warnings fatal but that's probably over optimistic of
gcc.


Robin

-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 252 bytes
Desc: OpenPGP digital signature
Url : http://lists.secondlife.com/pipermail/sldev/attachments/20080131/16beca92/signature.pgp