Skip to content
This repository was archived by the owner on May 30, 2018. It is now read-only.

Razer support and first-time device detection#21

Merged
Archomeda merged 2 commits intoArchomeda:masterfrom
antonpup:master
Dec 31, 2015
Merged

Razer support and first-time device detection#21
Archomeda merged 2 commits intoArchomeda:masterfrom
antonpup:master

Conversation

@antonpup
Copy link
Contributor

No description provided.

# Conflicts:
#	README.md
#	src/Config/MainConfigFile.cpp
#	src/Config/MainConfigFile.h
#	src/Devices/DeviceCorsair.cpp
#	src/Devices/DeviceCorsair.h
#	src/Devices/DeviceLogitech.cpp
#	src/Managers/DeviceManager.cpp
@Archomeda
Copy link
Owner

There's a small issue regarding the detection of a first-time run. The code doesn't reach the part that says that there's no configuration found and it's using the default settings. This error is my fault, and I didn't notice it until now. The code basically checks if the directory exists of the config file, and not the actual file itself. But that directory is created somewhere else in the code if it doesn't exist (I think that might be the logger).

I have fixed it locally already, so I'll merge this and commit my fix afterwards. Thanks!

Archomeda added a commit that referenced this pull request Dec 31, 2015
Razer support and first-time device detection
@Archomeda Archomeda merged commit ca85f78 into Archomeda:master Dec 31, 2015
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.

2 participants