From 36170ed3a331be3762a98572c6115527f177490d Mon Sep 17 00:00:00 2001 From: SANDESHSOBARAD Date: Tue, 21 Feb 2023 10:17:58 -0800 Subject: [PATCH 1/5] Resolved build and lint issue, fixed issue #1 --- GameDie.cpp | 3 ++- Roller | Bin 0 -> 71075 bytes main.cpp | 5 ++--- 3 files changed, 4 insertions(+), 4 deletions(-) create mode 100755 Roller diff --git a/GameDie.cpp b/GameDie.cpp index d8ecb3a..4790c1a 100644 --- a/GameDie.cpp +++ b/GameDie.cpp @@ -29,7 +29,8 @@ GameDie::GameDie(unsigned int num) { // generate a random number between 1-n where n is the counter size // (inclusive) and return it int GameDie::roll() { - int roll = rand_r() % roll_counter.size(); + unsigned int rolls = 2; + int roll = rand_r(&rolls) % roll_counter.size(); roll_counter[roll]++; return roll + 1; } diff --git a/Roller b/Roller new file mode 100755 index 0000000000000000000000000000000000000000..407602e468af1c4d07f2f5b775078e902e5b8ce8 GIT binary patch literal 71075 zcmeHw4R}=5we~)f2~7A=)KsZ}gc>zy)FcE5Tw0?cN+3cC(V|j^Fd1Mpe@-S4l(w<4 zz4&u`iI>k>r7b9J1I2#c)>_g_ue8`+wAzB@R;#E1sW%9=*lHz~l<$4_S!Z(2nVDoj z`+a?$@65wt@AsVjv-VnRuf6s@`|SMsgSY=SLMcPx=fia-t_^;rzSK5>x75dR&B7H7 zR$X~b`HkiC=kc!dr!zZmB_Njb5)uZ3)#VGTosV_?Hs?JhI1{(6iz$M^ShzLDnE7*; zxA#i3F+G#b=frGR{O7kyHtz(3&9Rmx&GEs`t~Sg2i##W`|J-EoFwdwzr zg);zoQTC&7fp3jAu>vPA??J6B$k1lyM@;{Q;QxGyGay~2-d0x3ol`!u!qT}NLRT9S z@V7(%uO7Y53|YKY1@9E2Cep`rJprXA3GUgY)Z=IXc{3Wcu3TKHelNWWUL<*V8t&!bqMlve(j2|KF0$nE+EsP6!2FHDMLM?Muxrzk zmrbu)|J|}PUc2h9NkzDqgA1hLHxiMNI!@jzqd4PY1iF31_~oH}EGOS8Sl>14EkK;l zv_9l!qgxvNw1A$_C6`=Qco~R_EYWn9*wd?v64EbN)f@}gUv`yb(KTI&|4Gk-&na~j z;Y7jYFMs23@%hm+4fO*}2=977j{_bDJPvpq@HpUcz~g|&0gnS72RsgV9Pl{calqq% z#{rK69tS)QcpUIJ;Bmm?fX4xk10DxF4tN~!IN))>Jk6ohFI(*{&!kd{+kaMD*G!Z-UfI%Qsxx|!=iL)0&N-B! zwy?aO3}efo(^YTJIjZYQq{qkL*&Et}$NKIl+YEX;c4P#9tIIsSw0Ab@Ekl|U?F#KM zHusgO4$|7K=~TTh;~C5ByW`5uNTc(f*vYiM&3(XZ=l6mMws}Tk+xqT!YV&f?brY^s`PxsM_-X=A%BTna&3)i6^nZrmgF7v~hx%md z%*@Rv%4|AwR%!1~MyPd9BA;6iHkOUwru6`A-dGmg_8&+?x*YOy=}sr~riVP5r1s2G zZSy-(2I=o_Yu1*)=Su&r2lB;vDeHP5+qbCaC^swX*@q^6ZrH=rG4=uWkJYFXdZXKc zBP6R#V2EWzFXC8{{jg3~)2pV*~q-`@~+5Z>LmNV>qX+f@=R0VAfi1W%$ z|M{xJP;K|?yt+aY<`7vgz$Cfbt&)CsXEG5?C4mAW?3TJLHLz~ZmW)N(EVrV z@egrXItN{L_9JP7Lk6}oDMOY++nAIG?I)pc$?8gxf2imSj%7*ITf2=Ey`` z+Dpo0J}}!&%s#fzj?tdkb__c>LinW9LsnPWo_DsNcWf5$e%Ot-A)h1Q$zjOrE!eTn z_M4BfFEig4vqs)qmNoodKgJE5(}MooZ}(vT9JalabfKO@u(h-U0oV`NpWZ7hIzs4s zsJEOL^#lxcIL8{tQTL4~pE`pxflNI@oMR}*JK7o6zaO+7ppJoFwx70~dhMWVC;H(z zuqmjUI{F66IEei5vC$@!&&4##mX3@UNUAA3!pewW|x}Q_t znbg7aj_I)`k4PH#%uZo-QpMU zUN>~FWH@Aja;UR;nY1A`y>_*MeUCC^UrEJZrit6_6t?p4Ftu)jQd6|fqTb=4f&Pox zKjHhm_xay^5%%Ar9X2Cd(+-}XUuplBdY^@ScAqr>{{Z9InLFsy4I(p>{?zia+M3W# z7f;BKSJ0k;^pm+Sa#-7=vVhn zbL{^H(vknHBZ@k-E}@+Lc;0Sd#NpSk!?T!T4>Py6BB z=alwdhWrpj~q^+FfdS#3d__L0voX1@R?{iBf9t|wZTz*EkDEZSF~ zUbjDQ_1RZIuS32&Y43+%e$y|fjqMnlpr4PR?hTKsJ^PS{eG3~{Uf{!Vyh0KMp?I?GJ3vM(76nA#DoN&|c_qP>)mefrV!96=$qzU#69`lb_Im zLr6CpeRm>!Z|dZulAhySC*+8;*s`g|@a10!d@k$Z?lQCoeA^gOd)^1F)NiI^dlSa9 z6!wJUn&s0`_6Jb@8{p|@g+9CNPqb5ZnV84+%g(a5lwJ3u@J%tsERp%7U0yqEb{9&I zw!fa?GKfa=HaF2vJTu6UQK{&EJRq)5e5A z2X)pkN_%~o=nLRM#ssr1d`@ftWBgdhxeaj+Fxz_v?lBF06pqpC-(j>->mmA4vpi=W zj!PUf@(k7cL(pdFYP+P_iqJ9NX;1D;ewXP{wxe$)Nt>4KOv0aJa|Djer`1o4Z zKZXSS^vO71wT-{_zKuRbefcElr0?B{ytI|{uPuL=@@GHf_<%W&V;w?4r|iB#GGF2X z=&IX>vdrxP+)pAO(nJ60X4K`-mjTjp9_2{eo~Tt2$+yjBU!hwZPaeTI^#;Z) zJud?T4Ooj5z>1s%lh6F8Imu26Lh(tg&$#?rp?9oGGepE`5q4%+L8x z$Xu)V6EN+um4~uGI_)oQ#`tUXEA}^zG1`|#U!#n;4$E~Trl(HY?Qk-^O=ed}_sQ55 zuBi^Hqu|GP(oFZ5J6)1KM7s4@hc?r`9{|rR`+iXK&z#qh7rd`9HgK+%(5|w6yA#*N zB=x!1E3~{sE{;BI%WL&9(r5Ljq1tzad@xVO`my$zlv2CZ+JcU=K;Mf)L8Htkbl=AO zp%?e)L+iN%>QuOQCwz_^r04y9|H9TUZFDe=^0mG27Rv^|=-+Ydu=+E?qZ8I(oJDxm z=+fRlA|2NX$Sc~$L(p^1Ro=vXOD9(%?zWHMvz}8?e;oSYq9wkDPWuDiBwe$y<$*3A z4g3puhOu4Sd9){92I^!POLW^zK2!G{GHFk=tDBJCL8B{;<~?*Kl?~F*jne-4p({Vf ziY)jrjyh~@qD-u@RQo>C4%#cOPbTPTlJY6bv*T%oL?j zm&%$wY2-LfADGX4(D`|ye-0lw2~WvS`T?Z-(D|wZx~%6??1wXOU-wJM_opdvINYb#Q%RiP)VDu<6wQn}JzAG3Nrcb_^a|Z^sxb7TGcUtGU4R)!gfM zR+;3hef@4a(%XF*$VabN$K@oLa*W@z%Z_b9nR?Cxx;dUvrzk`6r^gz*fbX&928dI4 z$zRGf8|kgNHF>-YH1*pg+uCpv9X8MeGif`D zdE~u#KjM&|tL@s(VC+n=L#}j`?+2*Y!N)RF7pRwU*|*z#8pztN%l}*G*1C)~eHYl$ zJFWr$yB$ORkJ_>26G5lHZoAKUeXtk&0DW(RS4W^bhtcl0^qgPTq`_aiPJapc9eztH z8KGU&SJFEj^TvyyAGDhrm#IB@i1R+QK0cqPeR;-`AJXjyEe80nWdTfI&4A7`^krh+ zjprva&F8S2*~!MlV$h25D`9@G%S3x0L;i0csgt(-muhqmWj1BiSLKqqh4|yfbX}egf|Y`nTiI;)>`-fGUzvLf$Co`LA$yz zS7u*dh4#}%w_58~xW{(|S=u)>ZG;{R=%0d~8_^%A)0&q^OIwTgXD@t5kCirCu`_X9 z>JjwKN1;!8eM+|t>v)fv>v)q$56Zocx`%Q}(<6{0*9>N%?OGm4TZVLfcl56z+2?*N z>!0wPX?1$&=_k?tDU{#NM4Lh%)ID}!2phQf|qJlFA~g&h;D_^hIi{jg!xdV6VeQ>^;!y$B_RA@0+s9gpT0-R2SQ} z{~WxBr%w->UdDPY`!w&M)$KbdY?^A+|5*bzNPdIag|Tj+OfkV}_dOSX-L?M^?L;|6u52YHln#cv&B ziz_Ylf%P!VGBEeQ6Yp@a4u7T5HvfFoqiq@6KTe%>H!#b+N5X;1WjgY&zh621++_QB z{$6P@|I*{jk0@)j#X)x&?Un3_(PI+)!;JCAxGuwfO&^Q= zjx=TF7&z!&6iLT2`u!HCZd!Wr5BdJ}d~;6-*J4O#G9Ebgge2h|-`!sB=HJfLV-S5? z&K+;ic~bXj_D}i)_oPv_9^0UkDd*@SC(BMa_LjK%!^yr2et~QEsGEAMADBDu?gh}kC;EsqjtXk z&BD7WU;O-*4LO<_8Z*GcbK4GkLiq)h911bq%C%& zgRQmKg);Buz8CvFj&GP#tn0p9>c70S>mYna&KnLu_x6v+*mV)sw=m|>2cLE01TGwVub z`)Gp#M(H%nWzOu^mkj93Na#z3xu+Dm!nO8C+s7T-gf@Tc&tprp4!}S97U$i1Y%_WI zCV0ztn~=rfjW(HcZun)iy|ewZmi!NQwvWNM{Mlo#Ab?kKq09zoizd=7mj&3s?IS=yt-XRZbvc75#5Fgkk9SH1tneDG6rKG;LodH>6EY3cG=giA{BXFRkGaA}`r+)yVWgqIWzkB1nsrjvoL{bG8Ny?IU+eg_(D4j&zs4n{ zy+1==`;T{%`_uNZa60=ubl5S6Dl>f>J-%j@cJ=GEvtRHzbhmdq%Aj9ubd+{IiFl4- z+5q~MhQDq8^|+_ky2h%r@;b!6;8_6a@yE)}a`Y?MlK8%S=n(ri`j@WfW&J)~>@9)zSG5b><+qwCk|i7&Zm*K z@nYw|V;p@Ot~Uk>t+|M8yi1n%3N_6!H@{-XH0~KDe;smbcglxtX+pYNa5>t7bk>?W z_k84mcCKe|ew8=UoF{=d<{tdP==mq?H=e3GxbKkkXx|O=eG>djpf6cjsMpap(A+=g zvGK*F7xcm0`+G62XkE7U_Ubtf%AbPx9rp(lF-jC}y??InQcl@4>=D+*i++;M_ z>gRhF%`1%WY|{?$&uBYG>o)4SWi~PQlUp{?Y`cw@?TLEAxmaLP>9l~c>Q6mGtv)oc z??f6D^w_Oi?s+Wbc_yyG(xvB%CDNW&z9jlX+d}(ATWa^iOnou!40S<|@%lYnzy2h% zDI6o+Hif)TMW42>AG7Hi<&SrhcpuS)wW==o{9PDRZnbQhrIWBh=3dG&=RAF5^Y{|# zM`y^~Lvk(XVg0m^)Lo7h)QyeJub1e3B(QgS{6P9fqyar*kNf8rx}6w%`OV4(q~qAc zJi085#d*LJ{+!%rLYl`XEHu}L29o{$@oF0B+&_NDG;J5hnRZcOycjT_*y}(Jcm>+A zSMp-~iBgB2jRFm$d??R1{%;?J7#6AzCwvQ>^! zIr;Y)YJ=f_+Aua5xz-e|QT+718G%Y;CRWAb_bxET=F#=B*##-AWiI)pnFxoPBXH5nQ+?wHk zE(3pJ7;Jed!;feCGUCK&9#NLN;im8QmEsQ(Z_fy<$uyqM*aZqriiHnj1Jy<@+7*`@ zytt~I zb1rJQD2IRYI$YbFQ(w~>sc)&zX>Mt1YK+FhwK+_uDjGtK(P%gn%Lz5s*Vi=EV$=22 zjnVp=*z}ysE}!36R~L@v%xL)3Fm-8?-!OG$bXiM%xFOa&Od(ayr8)D%x3xr~;o5)7 zsmW<*sb7NZg*lDUoMq8)O$;C!ZilH3S#9r7?Pi0@L~P;|~!tTCH+)=u-vCxA;z__Fjn{b9exa zS3g;a|L`N`eg3`gePv1=LHO<*{4HZFs_d>rmqe)MDpi7T#5|=IBkV*NLsUW6Oq46i+1`DTrR5YAA>@-vh= zeul~@JyRL0&s4snaLlr^RmQArWh{a=w;^1Yt^Df|-+Q()da!|F>p99dYBbvYab=8! zGtv2Rm2m{|5$7tkX^b*urwcZ=PO^&WlD{`Tp7ieLq?Y?pC6{Lpa9Ph9xqTCd4&*Akuvh|lA;Y^ zC&EpWl(7vjCH5gaI!XB!LV;HoD`R`H@;_ayd{x-6+W#8`@$xsS&D8BtZ4jx(ehDE; zjUhT6V*a5+h)#zPoetS=SPn#{!x0ifWEw+cIy_xMh)!dOPKOYg4)IPxhY*zxKPDkW zrZL`Y=n$gQAw;J`h)stOn-1B3Ne{%PL(Cm?2+`>=>xaCpQAxR&Ga2pqs| zEHn>|=$k%7_5A|ZnxV?T;q86?0Ls9%BcS?@;JQ9b_5BLhYr|CE`r)eYgW)&~0S83< zYJ}<=G7<+(j8uK!L-;bnA*ZRnvrkh-F0O^RK96fFu3zE$09Q5+kSN7fiE9zAI$XEo zx(6o|1cTQ#tc*0&2Iq&Hary}GjdSNLh)oIx3yTVZ!4){pq%j&?Qqvp`@>G+?hKkB1 zh2`aZ7F!-|Tp6qjH!O=S4~C=B#%TEpd;TdrQl&0Dtp#~`5J{|j{@eux!J7;Pu z(gDkwV%BRTMHf>9j)KtUSQO{6RD{rUHxcDdc7(u=DM3&biIvw|tvdCh+#Hx>($ffn z!Zr2D2`Zn|)DkOq)SXtg$xG{6nwLAdlvc(mJliI^s;sbl{(?zCyA)0)`zbZGuDRSH z_Tpf$xv4G^3odC{x-=ZEh(MLy+FDo=L?3AY3tK|5U`;F-Zm2DwXG(E z6MTZrIN>MU5DKSYI6F+R(AC)LV0A&TynMmbVAY%j#X)-olh7;cB1=L|O~FXBNlOjX zIJ}~!t_3Fs)jDKQ9BPC(ZR8b$-nvtx#8%4W5PJ=JQM{j<)=sT zCI=q}$~2e^PBL#r73>g29d4*uQWp+JmR2lmjuuTtPXvRTnhK`|mo`RM)s|wba2nnTls>M4~n>C_<$+ zDGX5j6r34FlbR+qU}#NKlUtSJCU_ETP;(glqNa8Ux(80Z(s~!Eg?2~7dWdjnT|DFD zP+hntT7HX-=(^?#>buJt!|)~R1|hO}!4z}?ZIKqlrh)(Ukp}3w9)+qa+*Y48RbQX% zZf#wa&eO1xObLcsYl4>I1$pRKQ!LWhU_Dt9UKVKx;;^vLa_bg`xw>FWL%6jG!)3VE zx(%i|(=CmL!-)lICNa;l*4AKCINIFUPyn4LrtqxFiQSLRj`>6 z1hFWF7Zr>}Fml5Y395&N`qkt2!b{00rjc|xr(!vHn4nvB0^cNxpWooux-F1%v z?`QYjNbkEvObGMxiA&cjUX z8fhEqpMj`@kD%Bgh3XyS?_{js@TXy<5BWcgE&ii_0CA`_Lr%Gy{bh*X--d$_{r(*f z@&E2$^8ZacwIrOz`aORVL#(!B`u&~XWO7)aO_?9Das495+ri@t{e|kU{!_Q22Z!{V z$hU^r?cY(=|o}+Pvb*+`Q4bGxBHV0Bto!Ww+*)=Huka-0UYXbs2S+ zu^zLS84I(E+l(E)S@{d|^Yc+XD$g@(XZ^)vH3gc0qZzql@5?M6U0au*J$lBBy!^@; zRoNsOwdbMK-0bDKqw{daWPS;otAaR4?KHG}{Z%kkz;_F-E;I2D1h)!)75F@(O>GkV zo;+VuZl*sU2eR>bkKj)Ov;47Fo6qkMJWKFnf_DjiN$@_w?+f-{W9B~x2feYp(SlbC zt`@ve@EXCt5WGq7B{(3C`Hu;X2_7-aEdR%XD+Q0lL2^vLSnyK8_X+-@;B8iZ!Osak zC|FgP<>%l)I`U_m;Q7ETziPJm{5yi11pl|-2L%tsL3T{PUGOD>cMA>+9yP}-k7pG! zeTm?Gf)@$KG*mxt5_qQLGw`IpZrro z_$z{6ns4HtNqYYR6ZZ+u6I_e~0-1k?;8nmZzg6%qdA><-77h|*`rU%(0yBMfwOM|f zJTDOZQ+fWh-~bLFWcq!AFBBZO&dfhkaIxTf1TPf4Q*fK${}H@S@HsfxkmdIXo+^0k z^=A1u2`&))px|o3uM1u-coYsuWO+@3OM%(`rvZ~zZTpnm}hA+|8~L41RoQ8k6@l!3PDuAb7-$=JU4&mkJKxphuQ} zpWvy29~T@G{F31R5PVed`@rN!_DyE_+)tNAU7R zW_jNc{G8x!!L6S)pXcI0O6Ff9I0(%8*9-nPd7h0kUdf+d3Z5Z&C=Rk@{?&r770feX znf~7emjq4x2f>R4UxEWLng1@qw+Vhw@MFNFr*g5G{}p*Ys>Z}u<(T<%1h)#_y~KR} ztl$wL6OWx>rmq!zi{MtlPYT{Dc*rNr^iPD%{4;?`PsvgfUnjU$@KQ;?R`6QE+Xeqv z@KM2k5!|`VEN{;FW_`N^uM&Jz@OK4gFE`WwPVfxDeS)h5Pt7&UTPrv!c$?tw3En68 zO~FS6XI^0DA0094yFhTh;F|b@QenaC1#c0& zOYrl8d6qEKzbClzRuhl8$SiNQ;7<$QB)DDhF2PR;J|Z|X&&;3QV3t<~OnLLHV%GN~ zdA_~L#KR|=>5mG|6CAkBe7;C9&p2lOuM2)j@Ls|GsQG;S#b$Y<1vgV`!n2zrw^b1Y@m1KfggR&!lGhuL#avY2u$-=>;DUyhgB5gz_`Y^R}7)EWtc? zoA?63Jb#|cCY~dh=W!F?ESTqV6W=B{CipJF>jZyQFwg5|`YnQaZa48y1n(8R zM=;Ov=JVeR=6T-4`~#G156|@`9zMy$Jl~u6Ji$EYo48Og&-*5xDVXPe6Msf9&;KUA zMKI3+C;psZo(E3s!*x0$kC&Sm=1yb3;HiRl&orOUveK8C_y)l|`&`#2m}jCBqgs@Q z@weKrUS5q0f7^v$b>XvzIP+iU!W}M* z*LiGDyuKG*_>V4}InT>Egnit8O*f5P<|u0P}Y3$FKZ9mB;w z`T?%L;yRA257!A?S`i>AAFd2seq8%;I~l!i;{K`7{0HQF5!WAaoecf&BK>=~25ZOP zkS0}I@JQug&HBc<)>?4V^=5klYw6j353lyj7XVf%xM99Sj^8M9rznyXC%5#>TFu30 z_xpwa3vYWr#G-UR-G8SA>HZ4RSeWil{9zWT`%4*Mq1t=>_YeH~@04F51cQldO8nL2N42fQ+vK4)xeUCip45^o0k zFKnCilI@~%sHCp~hDJ_NayQ)R!9s|?*Vz0Udz2i^tSp31#u|e)wYBW>GzYK>&|W(Yg@YR`cj-cX z8aby0x#2k*J5`rkm+!kXcarsCTI_a4*2be2sp=-1afou{<&x%MHaD;bA@FpYak!Po}k zlaoYIVX(~FM_!Gcp3T@Sj{TeT6VXz;;-+OTro}gOlTsI{=Vvf=&fRF~j;K19e;|O1m zwn}TYzhSxNmcP-TEK~=k#5<}ycSt(kwo@CroiXLdcZe_^M~#VBEw8~2DB!YeCg(@s>S=ME+E zrQF%K2N`w~^T+v`WEf69@F$j0o@C@t%h5t+KT5)^aaIP% zVvHPdcLW~|g>Kc~POXga6H@C-o8Yn*d?42l3y1XwK@PPqH|Gr1!77Y3`21%+E?XC= zEasO}`on@1^Ys{PpL-7Ao{(knNi|e9-NizFW)Nw>*Ip4!pu?P1#XkgVb`GA!T-B+V z&kxGx!9o`%uJhoFCewV(gJH5YDlKuw7Z6TjNO4R+tyy+DHrzc!KZ!aApJt>(K=M+^ z)~Wb#H@qy0H6X2LP@{?^dNIuL(Q#TVO&D)17NtAjB(WI-sG$C;uwRv|?+Kf+aEdhv znf?k10~ABn#5K8jIX*DHHC<(hG_@IvUij!0D{$btxeA3?)2jZ?jtPu* zS~04@W$pJ{MRf+BTbjg)w=21l3L*{nN4GMm1C(xwZdW2@i8g0qP9kLkc@vW@b6_(C zr{K_H5m<9O!JViq!g$zkjA3z?;X{4%YgWBFEv^ARX@$*-;D;h;RV~w7RbA*XQ?3t{ zlYZ4+jxVdLuI7}F&X;bPwbdg&h{bO>2af@CKAzRoi-A&FWu9b5a;)kOp7$ScjdpON z6W>~SE4sG6_hH|L4Q}EbldUD*4i~SQhs|70K;C2;iX}p%qIU?lE8BHPzI@bE)OI2RJ4p)kzd|@}|k2 z5*&&Uceiowf_X3o&ji5WvY>?jIFpO-%qVyH)k%v)U?<&>p0kvi8oAA{Ceg7X`i`RY=3T3WQa zUhcE?ljFzM;FK6JKSe9}>O=S31fHyn)W%##!K7K{_*{~_E!an~~0*hcx|toDLN?&6p>pVuP*Y*cMen z?oAPCZKmLXRH(}S^4je!&Ow^0ghiNBQ&qm8;ZCHcM$Ye}PWy6dMPho7LwxEQ8`y-E;Z=bKj=_9Ia-TR)2VL#V@8sP zgO`YbCfwjSm94V>#63>qmEwT7Qd`;-4zOXQ{w8Emqyov2D zN^JO~#8i_KQxzwsDlSeSEJ>557W0!F{rljx5j*hZ(GLLd)sg;X;~u`l@_keJ@R6KP ze{&ba6P#3_e`hM=%%eFzHA3(pL7Vm5n2+P6aLmoyFFrl%x1q~FcK6zMeJ|dB+UdKt zPq_b{|8>{wEq7je+t;3cs{XMrZ2kVi#$Oy-y2JnCo;|nv<}J)TvHFSCw}0v7pWHj^ z?)>k3*yn5SyZ`&={Bhpm${&td_~p&7*Ia+*x8MKC`Att<_>BiDtM^2IJiFo>=MSBD z#zimv@`rcbbL;FsjH;Uc{mtVZ|7=}%)_DIj_rCMBr_R3To6SqUIAQ7=r5~I4LCS_22yT=!Ktodd$~e z|8>#C6Hggi*ZusieOHt%~(p{Ob2t_dLIN^7mKG+dBJ&m1n>7U+eGw;F8Vvzw+?5 Orh~yZE^8k%@BaW5+WsQ| literal 0 HcmV?d00001 diff --git a/main.cpp b/main.cpp index 65055f9..39efd65 100644 --- a/main.cpp +++ b/main.cpp @@ -7,7 +7,7 @@ using std::cout; using std::endl; int main(int argc, char *argv[]) { - if ( argc != 2 || std::atoi(argv[1]) < 1 ){ + if ( argc != 2 || std::atoi(argv[1]) < 1 ) { cout << "Incorrect command.\n" << "Format: ./Roller \n" << "--------------------\n" @@ -15,8 +15,7 @@ int main(int argc, char *argv[]) { << "--------------------\n" << " - Required; a number 1 or greater representing the number\n" << " of faces on the die being rolled\n"; - } - else { + } else { int faces = std::atoi(argv[1]); GameDie die(faces); cout << die.roll() << endl; From b9a1afb19721a4a6e1433bda256ae7c8e5073964 Mon Sep 17 00:00:00 2001 From: SANDESHSOBARAD Date: Tue, 21 Feb 2023 10:30:47 -0800 Subject: [PATCH 2/5] CI workflow setup done, issue #2 --- .github/workflows/actions.yml | 23 +++++++++++++++++++++++ 1 file changed, 23 insertions(+) create mode 100644 .github/workflows/actions.yml diff --git a/.github/workflows/actions.yml b/.github/workflows/actions.yml new file mode 100644 index 0000000..e56ba07 --- /dev/null +++ b/.github/workflows/actions.yml @@ -0,0 +1,23 @@ +name: Build C++ + +on: + push: + branches: [ main ] + pull_request: + branches: [ main ] + +jobs: + install: + runs-on: ubuntu-latest + steps: + - name: Install dependencies + run: | + sudo apt-get update + sudo apt-get install -y -f build-essential g++ cmake + build: + needs: install + runs-on: ubuntu-latest + steps: + - uses: actions/checkout@v3 + - name: Build project + run: g++ main.cpp -std=c++17 -Wall -Wextra -Werror -o firstIO \ No newline at end of file From 8097923c2b06e6c6a66a103060d1651aaa39f187 Mon Sep 17 00:00:00 2001 From: SANDESHSOBARAD Date: Tue, 21 Feb 2023 10:36:27 -0800 Subject: [PATCH 3/5] Badge added to readme file, issue #2 --- README.md | 2 ++ 1 file changed, 2 insertions(+) diff --git a/README.md b/README.md index d46f7c6..c66591f 100644 --- a/README.md +++ b/README.md @@ -41,3 +41,5 @@ To check for your program's adoption of the style guide, within the docker container (see above), run **cpplint**: `cpplint *.cpp *.h` + +[![Build C++](https://github.com/SANDESHSOBARAD/Roller/actions/workflows/actions.yml/badge.svg)](https://github.com/SANDESHSOBARAD/Roller/actions/workflows/actions.yml) From d5c856956f1157e8231b16cfe8dae8dadd502bbd Mon Sep 17 00:00:00 2001 From: SANDESHSOBARAD Date: Tue, 21 Feb 2023 20:52:00 -0800 Subject: [PATCH 4/5] fixed action.yml, issue #2 --- .github/workflows/actions.yml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/.github/workflows/actions.yml b/.github/workflows/actions.yml index e56ba07..363c298 100644 --- a/.github/workflows/actions.yml +++ b/.github/workflows/actions.yml @@ -20,4 +20,4 @@ jobs: steps: - uses: actions/checkout@v3 - name: Build project - run: g++ main.cpp -std=c++17 -Wall -Wextra -Werror -o firstIO \ No newline at end of file + run: g++ GameDie.cpp -std=c++17 -Wall -Wextra -Werror -o firstIO \ No newline at end of file From 0beb44ac8f08bf628c2c10c0feb0968bcd32520a Mon Sep 17 00:00:00 2001 From: SANDESHSOBARAD Date: Tue, 21 Feb 2023 21:09:23 -0800 Subject: [PATCH 5/5] superlint and dockerimage added, issue #2 --- .github/workflows/actions.yml | 23 +++++++++-------------- .github/workflows/superlint.yml | 29 +++++++++++++++++++++++++++++ 2 files changed, 38 insertions(+), 14 deletions(-) create mode 100644 .github/workflows/superlint.yml diff --git a/.github/workflows/actions.yml b/.github/workflows/actions.yml index 363c298..afc032e 100644 --- a/.github/workflows/actions.yml +++ b/.github/workflows/actions.yml @@ -1,23 +1,18 @@ -name: Build C++ +name: Docker Image CI on: push: - branches: [ main ] + branches: [ "main" ] pull_request: - branches: [ main ] + branches: [ "main" ] jobs: - install: - runs-on: ubuntu-latest - steps: - - name: Install dependencies - run: | - sudo apt-get update - sudo apt-get install -y -f build-essential g++ cmake + build: - needs: install + runs-on: ubuntu-latest + steps: - - uses: actions/checkout@v3 - - name: Build project - run: g++ GameDie.cpp -std=c++17 -Wall -Wextra -Werror -o firstIO \ No newline at end of file + - uses: actions/checkout@v3 + - name: Build the Docker image + run: docker build . --file Dockerfile --tag cpp-container:$(date +%s) \ No newline at end of file diff --git a/.github/workflows/superlint.yml b/.github/workflows/superlint.yml new file mode 100644 index 0000000..ca69cec --- /dev/null +++ b/.github/workflows/superlint.yml @@ -0,0 +1,29 @@ +# This workflow executes several linters on changed files based on languages used in your code base whenever +# you push a code or open a pull request. +# +# You can adjust the behavior by modifying this file. +# For more information, see: +# https://github.com/github/super-linter +name: Lint Code Base + +on: + push: + branches: [ "main" ] + pull_request: + branches: [ "main" ] +jobs: + run-lint: + runs-on: ubuntu-latest + steps: + - name: Checkout code + uses: actions/checkout@v3 + with: + # Full git history is needed to get a proper list of changed files within `super-linter` + fetch-depth: 0 + + - name: Lint Code Base + uses: github/super-linter@v4 + env: + VALIDATE_ALL_CODEBASE: false + DEFAULT_BRANCH: "main" + GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}