search

Sdcross / MPX-OS

CS450 MPX Operating System
MIT License
0 stars 4 forks source link

Memory leaks #1

Open djbowman1 opened 7 years ago

djbowman1 commented 7 years ago

Fix memory leaks @Sdcross

Sdcross commented 7 years ago

I'm not seeing any more on my side, do you know of some examples?

djbowman1 commented 7 years ago

I just remember from testing before our final presentatoin we bumped up memory alloc to a much higher number to avoid any issues

On Mon, Jul 10, 2017 at 12:47 PM, Shaun Cross notifications@github.com wrote:

I'm not seeing any more on my side, do you know of some examples?

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/Sdcross/MPX-OS/issues/1#issuecomment-314165549, or mute the thread https://github.com/notifications/unsubscribe-auth/AIZzDRdYuAUf5O3iGywAQuhRzkfhha7pks5sMlW9gaJpZM4OTHDP .

Sdcross commented 7 years ago

You're right we did, but I thought we also put a fix in to keep it from leaking? Or was it only a patch for most of them?

djbowman1 commented 7 years ago

It was only for a big portion of it, if im not mistaken we were still leaking a bit here and there

On Mon, Jul 10, 2017 at 1:38 PM, Shaun Cross notifications@github.com wrote:

You're right we did, but I thought we also put a fix in to keep it from leaking? Or was it only a patch for most of them?

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/Sdcross/MPX-OS/issues/1#issuecomment-314179701, or mute the thread https://github.com/notifications/unsubscribe-auth/AIZzDXplKV-zVd7HXKPJlpgkMRqmDReHks5sMmGxgaJpZM4OTHDP .

Sdcross commented 7 years ago

We may have been.