Re: Patch 7.3.841: Innocuous compiler warning in "Huge" eval.c
- Tony Mechelynck wrote:
> On 26/02/13 19:36, Bram Moolenaar wrote:Missing "/testdir". Didn't notice diff omitted those two files.
> > Patch 7.3.841
> > Problem: When a "cond ? one : two" expression has a subscript it is not
> > parsed correctly. (Andy Wokula)
> > Solution: Handle a subscript also when the type is unknown. (Christian
> > Brabandt)
> > Files: src/eval.c, src/test49.vim, src/test49.ok
> src/version.c is also changed (by adding the patch number) but
> src/test49.* aren't changed by the patch. :-?
Will make another patch then.
> "hg -incoming" correctly mentioned only src/eval.c src/version.cFixed in 7.3.842.
> I get the following in the compile for "Huge" vim but no message for
> "Tiny" vim (which is not compiled with +eval):
> eval.c: In function ‘eval7’:
> eval.c:5173:7: warning: pointer targets in passing argument 1 of
> ‘vim_strsave’ differ in signedness [-Wpointer-sign]
> The message refers to the first nonblank in the following (new) line:
> rettv->vval.v_string = vim_strsave("");
> However, I suppose that a string of zero characters followed by one null
> byte is the same regardless of the assumed sign of the characters, so
> this message should be innocuous.
The average life of an organization chart is six months. You can safely
ignore any order from your boss that would take six months to complete.
(Scott Adams - The Dilbert principle)
/// Bram Moolenaar -- Bram@... -- http://www.Moolenaar.net \\\
/// sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ \\\
\\\ an exciting new programming language -- http://www.Zimbu.org ///
\\\ help me help AIDS victims -- http://ICCF-Holland.org ///
You received this message from the "vim_dev" maillist.
Do not top-post! Type your reply below the text you are replying to.
For more information, visit http://www.vim.org/maillist.php
You received this message because you are subscribed to the Google Groups "vim_dev" group.
To unsubscribe from this group and stop receiving emails from it, send an email to vim_dev+unsubscribe@....
For more options, visit https://groups.google.com/groups/opt_out.