From d298adc4e6f840191d556dddccaa76868bbca3c3 Mon Sep 17 00:00:00 2001 From: Michael Paulson Date: Mon, 7 Nov 2016 10:27:26 -0800 Subject: [PATCH] chore(lint) (#4079) * chore(lint) I accidentally let 2 pieces of lint in with my generated code. * fix(generate-code): I did not generate all the required code for tests. --- .gitignore | 1 + src/idl_gen_js.cpp | 4 +- tests/monster_test.bfbs | Bin 3320 -> 3384 bytes tests/monster_test_generated.js | 104 +++++++++--------- .../namespace_test1_generated.js | 12 +- .../namespace_test2_generated.js | 4 +- 6 files changed, 63 insertions(+), 62 deletions(-) diff --git a/.gitignore b/.gitignore index 79c70d0cd..9dfabefad 100755 --- a/.gitignore +++ b/.gitignore @@ -65,3 +65,4 @@ build/VS2010/FlatBuffers.sdf build/VS2010/FlatBuffers.opensdf build/VS2010/ipch/**/*.ipch *.so +Testing/Temporary diff --git a/src/idl_gen_js.cpp b/src/idl_gen_js.cpp index 23fd52c57..32c660c90 100644 --- a/src/idl_gen_js.cpp +++ b/src/idl_gen_js.cpp @@ -557,13 +557,13 @@ void GenStruct(const Parser &parser, StructDef &struct_def, std::string *code_pt "@returns {boolean}"); code += object_name + ".prototype.mutate_" + field.name + " = function(value) {\n"; - code += " var offset = this.bb.__offset(this.bb_pos, " + NumToString(field.value.offset) + ")\n\n"; + code += " var offset = this.bb.__offset(this.bb_pos, " + NumToString(field.value.offset) + ");\n\n"; code += " if (offset === 0) {\n"; code += " return false;\n"; code += " }\n\n"; code += " this.bb.write" + MakeCamel(GenType(field.value.type)) + "(this.bb_pos + offset, value);\n"; code += " return true;\n"; - code += "}\n\n"; + code += "};\n\n"; } // Emit vector helpers diff --git a/tests/monster_test.bfbs b/tests/monster_test.bfbs index 66d3ee8f215a67b4978d70df9bdb2e6d344e89b8..0369f896935ecf0822e50268c42f9fa4dfb9da87 100644 GIT binary patch literal 3384 zcmZ`+O=w(I6h1SVbY_ytPt(|rZIr2$5+sCl5=xLFF*a=!laywRE`-cW^2QmNc{9ws z85(hsg@_a>T@;ZbQY0%8aS_CY3lSH^g^M7?g$pUgbrw}qfBO8sd*6NYGQ=Cs+;{K& z&i_5Wixn5c`g?>+Q9OpvD6cQJ2+ z@}LpW5NKiZvQ@Fi=dM|mTG<|d!Pyk~!Sg)I_!W9-ml-GUpz*6dKD)Y5bsDZ+$4QU! zJq4R!sg+cezAeIcu$KzgBXKb565@lOFKI5IN0HpK`EV8INbi!|3|$ zvFFi0(&EzM945I6n@2Gl-* z%z4ZoW8Mc%fnvZP2d*OrCcTpR)AL494U8XD%Jsh)kN!U1R~Prks@JW}YN6rQOU|2F zkt@KTlIH?^6l*|xVYCIxqM1U{!AkGqttM;g;fR4MDBy_=Zf>C2sRpev2P|C95i zoX4rosq@~cy+@~$GdFx03@vRhQ1HIzrIA(R-&_AE|Ds%n{9CJ5%MHROfIlJG0DoW0 z>^bV=i&mr9JU_Ksa9Zaq?B#(!E)xNM2y1L7!}vxBe-r0V$asLC)%Zjh-<-?_^1sFT zu#5%xy2i)D_(ll7hx23dY=Hk%<9m$1Vf$gHWFC3mkc z*4R#j@y34Qw{Sj)uFP}ZudK$$!+2vq@n7P6K%NQkYZ@PA4KZI#6mdUezh5#z9k2YQ z>=?k67qo7LwJS9jEmmSU>yxJgnGSjy?s$wnfHEDBy-_Gz?pm`@Pz_e5!IQ#GC5Co? z<;%yT@P65jU7_qe_Iq)w4emC27Gw_)XL97%kjo~_OLhx9?BFIQkF`C09e7=Hz|TWw z5>=eRGkMr)QmCyss)Id;`cU@DF8VWt=Y8l%z$#-`OVzbzr$5ADk_`6acsm)~CmP2v z260?0^u$HcOWQx-;qZ?kzkTmaEOI-qT2A4=d)$IhIc!%G84GfK{a;-(#PGFJ zt2X3o@KGLP!x(b^4Co!u4ColB<6W}sTCNl!i@umRoYb<9==;!gc-cO{9_=-l=QSot z9WA^tSrNQ1>cxm(GjyC|0bfh=^)Wsr1Z_gqk3`0>CO740Kz#dbfpVZx&>m<9bQ5%l zA1n6yq>|Mgs^>4X>;v^R3I5lyAA`lFZtiJZPy?(lyuxG)`(3ce_?*?a6yrh+@8LG~ z!m(S?m|n*JZP52vM{tXcj8^De*M`31fF2l(o8kE^#B&Y%cd(}ICePG04>}9#$Zrm2 z9{L%XMo%^Qq8>9$zL*c*CFZF;*t=LgrjjLynduvYh%4ptYnwLuvLlqm{8Aq8Eaw_% z3Uo+5X?Jj^^tT9r{-BK!NHFo?cR03`Eo*p4qd)1dMxVi-$VWT0P3&8`6X`xI-HfB9 w&ck1w8c!Y*Z#+u-8{hm-kR3N4X7t>|fLN_hBAqg5%i-UuuR6_!`T&;y0O$Nw7oQzp>!cGT!d0e2w9ZkqKgtjR}w7t7gN};J~ZC}6d{>)5=xo|x9&H2u` z=kK2P91@Y4i!%#gSP4l=SfbJ^12Tvw!C#Rv&=lwb=<^Pd2>2#=G$!(KkI0j_$T!%J z^@=^%wH=WzktJxF6*G+^knCLK25g81ZO*`jmk;$|3BTGkez4 ze}jNIAZOXklV|+^JNzvQvWw@; z6lyM?e=_psxVrHBrHbo%CI;#<2ef1T$5x$t)y-g+fQ)@2h*#H+oPJM(PJxW?SNxn? znMVZ|%Fddbx>zX{-}BZAwG5r1?GX@9OQ;O#+~J3q_Aibz=)LxbJNjiTEI9?#1F_q@ z(cV!|OWthS2|tPy9*Xz+#)=hvaNq zy!D^>d5w>L2R?u)(bm54pZHHSKJp8AznoTh&J^mgUMv)GUu#^)z+XX-kl{KGqfg=7 zV%f*Yl`kL%>qkq=6!v{`QelVjOuMw{f!s{C;P|VxY*r0b*}|+tPlwCU=~X?b=L&nv z_1q$Dzc9A3m2K>MWrN*C|HE!Sbsj;7)7$2xMMvK+;7)6uz60Q6 zm=wgb|81X7V7m0;encSQoS}VOUm3-{j3P?nSx<}x6)%|FRd-hXEQ{DjaJz|PPHYZW z{Ef1U=cgLW6v}Pnr5kF*F;42(MSs@Ju#c;IBx<(rLca@>l341{e%@PmJ-<}hfX7kg zyPlEEc@oj_O>U#%JBXe!W~>n3R@2yA1~Dc&##yT8L{yz)V<9hB@}){%wS&T*Q&^E~ zCGUEf0!t|`b&MTJoYQDya2||9_E4Xr?vamuSk<)=FL8KH$?qDc`g{lbP8HYxaqPDxjq7~?Zga=znfH%C z>ssFP96W@I+PqQD2G#ROdiEf1mNAZRjqV+A+MEDU?<}aLo{Mn1?PnnIyqmWFSQqBR zE?XD&AM?(-*wwiAoI|`V_U7vQKVa?zznH0S?AB{YTVYU}8q!~N)-vSUKh6dFJ%At0 zqJJTE!*stfo>kCebq}PI6Gdm$C>TaP=97L=$5HoNPaHiNLL3 n?e~5