Eclipse Community Forums: 4DIAC - Framework for Distributed Industrial Automation and Control » Changes in forte

Home » Eclipse Projects » 4DIAC - Framework for Distributed Industrial Automation and Control » Changes in forte_sync.cpp

Show: Today's Messages :: Show Polls :: Message Navigator

Changes in forte_sync.cpp [message #1798440]

Mon, 19 November 2018 09:01

Wolfgang Trischnigg

Messages: 41
Registered: April 2016

Member

As I was getting errors when I build the forte, I have to change this line in the forte_sync.cpp

CPCSyncObject::CPCSyncObject() {
InitializeSRWLock(&mLock);
}

Can you please add this to Release 1.10.?

Best,
Wolfgang

Report message to a moderator

Re: Changes in forte_sync.cpp [message #1798441 is a reply to message #1798440]

Mon, 19 November 2018 09:09

Martin Melik-Merkumians

Messages: 104
Registered: March 2016

Senior Member

In order to be safe from the legal perspective, please create an Eclipse Account and sign an Eclipse Contributors Agreement.
After you have done this, you can push your change to the Gerrit code review system. For this you can either use the git review command, if you have installed it, or stick to the documentation found at https://www.eclipse.org/4diac/en_help.php "Developmen Documentation"->"FORTE"->"Contributing to FORTE"

If you want to use git review, install it, then in the FORTE repository type the command "git review -s" to load the setup from the repository. After that do your changes, add and commit them, and at last, if you want to push them to the 1.10.x code review, type "git review 1.10.x".

Please also create a bug and explain your issue, as each platform has its own forte_sync.cpp

Best regards,
Martin

Report message to a moderator

Re: Changes in forte_sync.cpp [message #1798442 is a reply to message #1798440]

Mon, 19 November 2018 09:13

Jose Maria Jesus Cabral Lassalle

Messages: 199
Registered: February 2016

Senior Member

Can you tell us which platform and compiler version are you using? If it's windows and mingw, check this https://www.eclipse.org/forums/index.php/t/1096099/

Report message to a moderator

Re: Changes in forte_sync.cpp [message #1798550 is a reply to message #1798442]

Tue, 20 November 2018 10:47

Wolfgang Trischnigg

Messages: 41
Registered: April 2016

Member

Hello I'm using:
- WIN32
- VisualStudio 2010 Express
- CMake 3.1.1

Report message to a moderator

Re: Changes in forte_sync.cpp [message #1798578 is a reply to message #1798550]

Tue, 20 November 2018 15:38

Martin Melik-Merkumians

Messages: 104
Registered: March 2016

Senior Member

Which exact Windows version do you use? Your error is most likely related to the OS API for thread synchronization, which is dependent on the exact version of your OS.

Report message to a moderator