On Fri, Feb 06 2015, Christopher Li <sparse@xxxxxxxxxxx> wrote: > On Wed, Feb 4, 2015 at 3:38 PM, Luc Van Oostenryck > <luc.vanoostenryck@xxxxxxxxx> wrote: >> >> Working perfectly here. >> > > Any update of the string test case? > > I commit a review version of the immutable string fix in review-immutable-string > branch. Compare to the last patch, it take into account that string > can come from > macro arguments rather than macro body. > > It also avoid the string copy if there is no change, that is pretty > common as well. > > Care to give this version a try? > > https://git.kernel.org/cgit/devel/sparse/chrisl/sparse.git/commit/?h=review-immutable-string&id=de1fa7e60d3d179a1b67c47a0429b2d0ac4e4842 Sorry for taking so long to reply. This seems to work for the cases I can think of, but I haven't put it to the real test yet (whether the format strings are going to be messed up). Rasmus -- To unsubscribe from this list: send the line "unsubscribe linux-sparse" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html