Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Darkcoin prettyhatmachine update #8

Open
wants to merge 37 commits into
base: master
Choose a base branch
from

Conversation

princeofdarkness76
Copy link

Conflicts:
api.c
sgminer.c
util.c

Sanjin Trošelj and others added 30 commits January 27, 2014 22:04
Cherry-picked by veox.
Cherry-picked by veox.

Previous variable 'new' was badly named, but troky's 'n' was too cryptic.
Therefore renamed to 'newsize'.
Cherry-picked by veox.
Cherry-picked by veox.

Not a big fan of modifying compat libraries, makes it harder to maintain
later on. See if these changes are relevant upstream.
Cherry-picked by veox.

Changes from troky: struct device_drv opencl_drv tab alignment.
Cherry-picked by veox.
Cherry-picked by veox.
Cherry-picked by veox.
Cherry-picked by veox.
Cherry-picked by veox.
Cherry-picked by veox.
…il.c

Cherry-picked by veox.

Introduces a segfault. Expect a HEAD reset.
Introduced by commit 87e5f35

The cast/ref/deref is performed wrong and breaks share/best share display.
It is not the value of target that should be increased, but its address.

If all else fails, add an `#ifdef _MSVC_VER`; we should avoid it if possible.

In either case, constructs like `&(*(((uint64_t *)target) + 24))` must be
explained (in a comment).
Conflicts (resolved):
	sgminer.c
Conflicts (resolved):
	sgminer.c
Conflicts (resolved):
	sgminer.c
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants