From c31b3b632c6f59e02b2dbb472aa1cd2769eda52f Mon Sep 17 00:00:00 2001
From: TriantaTV <gregcraw2001@gmail.com>
Date: Mon, 3 Oct 2022 03:31:33 -0500
Subject: [PATCH] Fixed issues with semaphores. Program fully functional

---
 DiningPhilosophers.c   |  10 ++++++----
 DiningPhilosophers.out | Bin 16808 -> 16808 bytes
 2 files changed, 6 insertions(+), 4 deletions(-)

diff --git a/DiningPhilosophers.c b/DiningPhilosophers.c
index 9735572..63be6a8 100644
--- a/DiningPhilosophers.c
+++ b/DiningPhilosophers.c
@@ -86,15 +86,15 @@ void* Philosopher(void* philosopherPassed)
     PhilosopherData *philosopherSelected = (PhilosopherData*) philosopherPassed;
     while (true)
     {
-        Think(philosopherSelected);                /*philosopher is thinking*/
-        if (philosopherSelected->eatingCount == 2)
+        if (philosopherSelected->eatingCount > 2)
         {
-            printf("Philosopher %d ate twice...\n", philosopherSelected->position);
+            // printf("Philosopher %d ate twice...\n", philosopherSelected->position);
             return;
         }
         TakeForks(philosopherSelected);          /*acquire two forks or block*/
         Eat(philosopherSelected);                  /*yum-yum, spaghetti*/
         PutForks(philosopherSelected);           /*put both forks back on table*/
+        Think(philosopherSelected);                /*philosopher is thinking*/
     }
 }
 
@@ -106,9 +106,10 @@ void PhilosopherInit(int philosopherTotal)
     sem_init(&semaphore, 0, 1);
     while (!AllPhilosophersFull(PhilosopherList, philosopherTotal))
     {
-        AllowPhilosopher();
         sleep(1);
+        AllowPhilosopher();
     }
+    sleep(5);
     for (int i = 0; i < philosopherTotal; i++)
     {
         printf("Philosopher %d eating count: %d\n", PhilosopherList[i].position, PhilosopherList[i].eatingCount);
@@ -123,6 +124,7 @@ void PhilosopherListInit(PhilosopherData PhilosopherList[], int philosopherTotal
         PhilosopherList[i].position = i;
         PhilosopherList[i].eatingCount = 0;
         PhilosopherList[i].state = THINKING;
+        printf("Philosopher %d is thinking...\n", i);
         pthread_create(&PhilosopherList[i].thread, NULL, &Philosopher, &PhilosopherList[i]);
     }
 }
diff --git a/DiningPhilosophers.out b/DiningPhilosophers.out
index af582e668d96240039c529a00b4a28b955f0cd69..21e4ff26629b38d1c9430a35291f43d84b9208d8 100755
GIT binary patch
delta 1005
zcmYk5TS!x39Kg?aHgmI@Zp>|><(V_)r8TFOm(|R2Y(;@VQj$>v!ybCjQb>C+ndNQ4
z>K}yAgCVBk%jCRt-%`>MCf(442!kFB!R+E%wfOp<on|}m{eS=4_rH8_hTUY?O?)yU
z?~^gnuMd>OEDuOTE5Pq3W}*snqWPcD!!D~=+ecek=knf%Om(9J<Y@n^PTIwLBpJ!1
zksnbNld@jM9cJ4KoxS$3vwLc>1*^vNY)WZNx5W85DZ?yHc=-}_8L6Jfw)Y8&$3lPS
z)r7e`1Xl9@rgTZ|7eb4#QNJt17nfLu>8+l|vJ2g_LR$JCO5H1@BhdWDpCDumVJ<vM
zJ)meziWU;@MIcpUAVDbA7|A<uX&N-$li2K;JTAxd3vbS7DmRZrC>6)S20HY*?Ge&z
zk67&=lRY{p&>9p+;xFC=m)4})8}F+98JqYx+|dpzO7ZP+TydCNP^-&G_8Y}TJHwQE
zPfYM#x~9bITs?GhR!Gt-z{fe;NV>#&1GiG*$@xra5E(?SAm1Wo_&jG;DD|Oz_zR^R
z+7FO>kjk%=o<im$FCm+e{m5x#0J-HGr3=W?RZ6A!td${3vyn~6O60u|RnR(A%70MW
zfoz3AeLnHQOTCsHgHL)Rc>&5SZCY>{7r+5xUj`~3Jh@Pjl}XNn3*`v3qa1+SD4)ZV
ztjsnkj#(pIu`5!pu=_TyW2-hK;wy3JI*4QK#<&{0uEiZ5iCZd>vX~`wXuup3uCN}t
zmLp+_P(+A((XabXX#whM@%m^BAr2=E2_y$j8#2iuxQvpAL6kH6Qv)xDDj5eCOeMs_
zkC~<<<RP?J9Bi0_am#F4u}-MSFjf)FMQo&Em5c|*3eDs!d?<8~bC6q9OlrVYWR!1G
zi-~IfT9KKgSu~;<og39j+)>4(Fk>*keTeMh+jn%z$X1Zs%yQ4aP9CVRIoJ>bJ+^I0
UaX;v|ZKvF;7t48hQa;4~1@Q+=cK`qY

delta 1039
zcmYk5U2F_d6vyYxE_SL`yQ$qRWoInbN80TNDk?>Ev-K@$n(!d$Myn)fX`2!eWPNmX
zQ4UW$XyRiVCEJV#JkXRVi9{kIAs#C6up($RD9ZJp9j%?@p5M9W-1|T0o@8QSiiIg|
zBjvnJN`B`#ll)4b#oYDjE6Gm|w`OktlTXN>veMtb`nZmEmvwYji2T<LRgR4|)}xl3
ziF8`;a1_(V@<?xDl~t0;TLVL_Qg}#i@+945*2tsslVhsWk&5)*HpCPC!$N$RWu%Q;
zE78D+|KpDkBR(Pi;j%tCYcZ{UhHG~T=AIz{y*qQ0dWrU5kjgS;pUaYT-#C=Gj%aJ;
zQz-RCUK%xiqhh$9j&wJ@;EqYF><7|P+DR}uy>yFbI(_tj*E;LD!a0=(oinE0#5J=b
zG^YN9Mqrp)1H&bOcN3!pyujtPZ&Ff}F7j4at~;0-3jGkmY_E#1yJGgM82;!W+BxLT
z$$029-x3`XqHXg7Nsqc4$8oo8J*(DwHf-CzqpqQ@ep}5Rb!N31YOGNko40SN$;->b
z8zkPUF5w<^A-AdOL7HMIIfq~-mhH0;kHHw&58eiC7*2V=5OM4a282+tzW^=+2f!+D
z{1+h(ftBDTuoLVCy@NvZfw^CWuwu}jZ$iuf*Mnu?xo^TQs^LujF2oVAl8<WhsExZc
z7cJr^nvdc<pt-VRpYTdFF!JTViDD_NP4`d*FNckA2sX@zVQ+BzbkD(eXuH|C60m0+
zlUGa_EAL7lhf%5stwCEQXg>{ANAtm`C2Y0)B%CrT)Z>hXCwzfFdPaVq)G+dG$PXf)
zpPH{trX}Qgsdo%nIN<frY_5Sl#YbT~^ebLn;k~(ws73F}?X}Pw-d7Nm-5U24yv#0i
z8)|6IgtQ+;9CD`A!d-=as^zzZLE6c4ii)Y3Lq$HNB+ERQrk^SDQ+B{<Zj(L3eT4Tp
zqztK(U(LTp#roj|$8F?erPQyq{HxT$%S(f@i}*xoURug8+*7(xiEE}@T&Jbi<iE~q
BQJVk&