Bug fix: In several string ops, I used a local var that just told whether any of the arguments were varying, to determine whether to assign to just the first element of Result, or all of them.  I should really have been checking whether Result itself was varying.  They could differ because the adjust_varying(Result) will make it varying even if none of the args are varying, if the instruction is itself inside a varying conditional. http://codereview.appspot.com/198076/show Review by: ckulla
Bug fix: In several string ops, I used a local var that just told
whether any of the arguments were varying, to determine whether to
assign to just the first element of Result, or all of them.  I should
really have been checking whether Result itself was varying.  They could
differ because the adjust_varying(Result) will make it varying even if
none of the args are varying, if the instruction is itself inside a
varying conditional.
http://codereview.appspot.com/198076/show
Review by: ckulla


git-svn-id: 93cebb81a0f5cc00f31a7ad40d16024532691b70@564 a3683778-fedd-11de-87c3-653bb6514e15

aconty/OpenShadingLanguage,fpsunflower/OpenShadingLanguage,fpsunflower/OpenShadingLanguage,yaroslav-tarasov/OpenShadingLanguage,sjparker/OpenShadingLanguage,imageworks/OpenShadingLanguage,brechtvl/OpenShadingLanguage,Anteru/OpenShadingLanguage,mcanthony/OpenShadingLanguage,imageworks/OpenShadingLanguage,juvenal/OpenShadingLanguage,lgritz/OpenShadingLanguage,lgritz/OpenShadingLanguage,amartinez-cg/OpenShadingLanguage,svenstaro/OpenShadingLanguage,imageworks/OpenShadingLanguage,juvenal/OpenShadingLanguage,aconty/OpenShadingLanguage,rlmh/OpenShadingLanguage,yaroslav-tarasov/OpenShadingLanguage,johnhaddon/OpenShadingLanguage,yaroslav-tarasov/OpenShadingLanguage,rlmh/OpenShadingLanguage,svenstaro/OpenShadingLanguage,rlmh/OpenShadingLanguage,amartinez-cg/OpenShadingLanguage,amartinez-cg/OpenShadingLanguage,imageworks/OpenShadingLanguage,fpsunflower/OpenShadingLanguage,lgritz/OpenShadingLanguage,Anteru/OpenShadingLanguage,rlmh/OpenShadingLanguage,sjparker/OpenShadingLanguage,mcanthony/OpenShadingLanguage,amartinez-cg/OpenShadingLanguage,gzorin/OpenShadingLanguage,brechtvl/OpenShadingLanguage,aconty/OpenShadingLanguage,aconty/OpenShadingLanguage,johnhaddon/OpenShadingLanguage,mcanthony/OpenShadingLanguage,svenstaro/OpenShadingLanguage,sjparker/OpenShadingLanguage,brechtvl/OpenShadingLanguage,juvenal/OpenShadingLanguage,imageworks/OpenShadingLanguage,Anteru/OpenShadingLanguage,sjparker/OpenShadingLanguage,johnhaddon/OpenShadingLanguage,mcanthony/OpenShadingLanguage,lgritz/OpenShadingLanguage,gzorin/OpenShadingLanguage,brechtvl/OpenShadingLanguage,juvenal/OpenShadingLanguage,brechtvl/OpenShadingLanguage,aconty/OpenShadingLanguage,yaroslav-tarasov/OpenShadingLanguage,fpsunflower/OpenShadingLanguage,svenstaro/OpenShadingLanguage,lgritz/OpenShadingLanguage,amartinez-cg/OpenShadingLanguage,gzorin/OpenShadingLanguage,Anteru/OpenShadingLanguage,gzorin/OpenShadingLanguage,gzorin/OpenShadingLanguage,johnhaddon/OpenShadingLanguage