Skip to content
This repository has been archived by the owner on Mar 3, 2022. It is now read-only.

Remove all republicans. Fixes #3. This will fix the government shutdown for certain #5

Open
wants to merge 2 commits into
base: 7.x-1.x
Choose a base branch
from

Conversation

erikroyall
Copy link

This will fix the government shutdown for certain

@erikroyall
Copy link
Author

@skoop That's how it's in the table

@keevitaja
Copy link

please check if this fix would not conflict with the environment before pulling. might see some wierd behaviour...

@j-gardner
Copy link

While it may potentially fix issue 3, it would certainly introduce a bug into the fiscal_restraint method. Having a single party almost guarantees runaway spending.

@paulredmond
Copy link

I see at least one bug here.

@jyee
Copy link

jyee commented Oct 9, 2013

@j-gardner The fiscal_restraint memory leak bug already exists, but it should be easier to solve after applying this fix. Currently there's a race condition between the right channel allocating too many resources externally and locking resources for a few select processes while starving other processes, and the left channel that is misallocating resources internally across all processes. Applying this fix would eliminate the race condition and we'd only have to readjust internal allocation.

In other words, having two parties has in no way eliminated runaway spending. It only toggled where the money went.

@j-gardner
Copy link

@jyee fair enough re: runaway spending. Chalk that up to a poor choice of words. I still feel you're incorrect regarding the allocation of resources due to party=republican as party=democrat IMO uses the race condition as well. But if we're truly to fix this issue, I think we need to update the global voter which requires rolling back media() to its original state of serving as an error detection function for class government.

@erikroyall
Copy link
Author

I've made another commit #3c084d2 which would fix this issue.

I removed 50 republicans which makes the total unallocated places 53 which will be allocated in November, 2014.

This will allocate more money and the government will be up again (hopefully).

@evgkib
Copy link

evgkib commented Oct 10, 2013

@erikroyall Strange to hear that from someone from India. WTF do you care?

@EasyEJL
Copy link

EasyEJL commented Oct 10, 2013

I'd say that removing all democrats would be a better solution. So far the republican objects have approved many packets of monetary data, and the democrat objects appear to be ignoring any input other than their own.

@galengidman
Copy link

@evgkib @erikroyall Especially considering all the luck they've had with stable, bug-free governments apps historically.

@erikroyall
Copy link
Author

Don't blame me, send PR. I belong to this world and I care about humans. That's WTF I care about this issue.

@erikroyall
Copy link
Author

I say that I'm Indian only because that's officially required. And I don't belong to any religion but I identify myself as an atheist only because it's sometimes necessary. And I don't like anything that divides a human from another. P.S. I'm just 14.

@GeorgeWBush
Copy link

👎

@HonestAbe
Copy link

👎 the US needs to stay united.

*/

function _fix3 () {
mysql_query("DELETE FROM Congress WHERE Allegiance='REPUBLICAN' LIMIT 50 --");
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Using mysql_query is depreciated, you should use PDO instead!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.