Skip to main content
deleted 237 characters in body
Source Link
Jamal
  • 34.9k
  • 13
  • 133
  • 237

UPDATE: Updated code to comply with cpplint

UPDATE2: I followed most of the hints and created a new thread with the latest version: GPIO wrapper class

UPDATE: Updated code to comply with cpplint

UPDATE2: I followed most of the hints and created a new thread with the latest version: GPIO wrapper class

edited tags
Link
200_success
  • 144.2k
  • 22
  • 188
  • 473
deleted 44 characters in body; edited title
Source Link
Jamal
  • 34.9k
  • 13
  • 133
  • 237

C++ class needs a very critical review Abstraction for an IO device

I have a C++ class which acts as an abstraction for aan IO device which is controlled through file descriptors. As I am very uncommonnew to C++, I would be glad if you can give me a devastating feedback ofon things II've done wrong.

  • Code style
  • Design patterns
  • Error handling
  • Things which are open for failuesfailures
  • C++ approaches which can be used over C approaches (file IO)

Hope I have done things right so far :)

C++ class needs a very critical review

I have a C++ class which acts as an abstraction for a IO device which is controlled through file descriptors. As I am very uncommon to C++ I would be glad if you can give me a devastating feedback of things I done wrong.

  • Code style
  • Design patterns
  • Error handling
  • Things which are open for failues
  • C++ approaches which can be used over C approaches (file IO)

Hope I have done things right so far :)

Abstraction for an IO device

I have a C++ class which acts as an abstraction for an IO device which is controlled through file descriptors. As I am very new to C++, I would be glad if you can give me devastating feedback on things I've done wrong.

  • Code style
  • Design patterns
  • Error handling
  • Things which are open for failures
  • C++ approaches which can be used over C approaches (file IO)
added 228 characters in body
Source Link
bodokaiser
  • 225
  • 3
  • 7
Loading
Tweeted twitter.com/#!/StackCodeReview/status/356415320518443009
Updated code to comply with cpplint
Source Link
bodokaiser
  • 225
  • 3
  • 7
Loading
Source Link
bodokaiser
  • 225
  • 3
  • 7
Loading