From: Borut R. <bor...@si...> - 2007-05-21 20:41:20
|
Raphael Neider wrote: > Hi Borut, > > >> is this something that should be fixed also for the release? >> > > The patch does avoid the assertion and raises an incorrect warning instead: > at 5: error 129: pointer types incompatible > I would say that this message is more helpful (line 5 is correctly > pointing at the problem) than the assertion, so it would probably make > sense to include the patch in the release. I did not ask to include it > just because I... don't know why, code freeze and all... > > This patch is very unlikely to break anything, so yeah, why not include > it? Shall I proceed? > > Yes if it will be done early enough to be included in the tomorrow snapshot build. It would be probably wise to submit a bug report too. Will you do it? Borut |