Skip to content

Commit

Permalink
Add line break
Browse files Browse the repository at this point in the history
In a  previous commit, this was edited to break up lines. A mistake was made, neglecting the line break characters. Line 35 is not a switch statement, but a define statement. As such, the line needs to be broken up as a string. I tested this, but let me know if I made a mistake.
  • Loading branch information
mflav committed Jun 23, 2017
1 parent 5897bc3 commit d7ba90e
Showing 1 changed file with 4 additions and 7 deletions.
11 changes: 4 additions & 7 deletions linux/os.h
Expand Up @@ -32,12 +32,9 @@ along with sedutil. If not, see <http://www.gnu.org/licenses/>.
// a few OS specific methods that need to be worked out
#define SNPRINTF snprintf
// Cumbersome, but trying to avoid editing common files. If /sys/block is not present, reverts to enumerating /dev/sd* devices.
#define DEVICEMASK switch(DtaDevOS::getNextDevice(i)) {
case 1 : snprintf(devname,23,"/dev/%s",DtaDevOS::getDeviceName());
break;
case -1 : snprintf(devname,23,"/dev/sd%c",'a'+i);
break;
default : snprintf(devname,23,"/dev/sdX");
break;
#define DEVICEMASK switch(DtaDevOS::getNextDevice(i)) { \
case 1 : snprintf(devname,23,"/dev/%s",DtaDevOS::getDeviceName()); break; \
case -1 : snprintf(devname,23,"/dev/sd%c",'a'+i); break; \
default : snprintf(devname,23,"/dev/sdX"); break; \
}
#define DEVICEEXAMPLE "/dev/sdc"

2 comments on commit d7ba90e

@cristim
Copy link

Choose a reason for hiding this comment

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

Ouch, we missed this one, @laser-beans please create a PR for this.

@mflav
Copy link
Owner Author

@mflav mflav commented on d7ba90e Jul 15, 2017

Choose a reason for hiding this comment

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

No need, this must have already been fixed. It looks like the master has the correct format.

Please sign in to comment.