-
Notifications
You must be signed in to change notification settings - Fork 31
Fix warnings #95
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
base: devel
Are you sure you want to change the base?
Fix warnings #95
Changes from all commits
d7188cb
280e0c0
0c8cc44
126539b
b5d6831
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -17,18 +17,19 @@ Place, Suite 330, Boston, MA 02111-1307 USA | |
| #if !defined(SHMUTIL_H_) | ||
| #define SHMUTIL_H_ | ||
|
|
||
| #include <sys/types.h> | ||
| #include "sheep.h" | ||
| #include "spindle_launch.h" | ||
|
|
||
| typedef struct { | ||
| volatile unsigned long *lock; | ||
| volatile unsigned long *held_by; | ||
| volatile pid_t *lock; | ||
|
Member
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Leave 'lock' an unsigned long (or a uint64_t). Same with the other 'lock' variables below. held_by can remain a pid_t |
||
| volatile pid_t *held_by; | ||
| int ref_count; | ||
| } lock_t; | ||
|
|
||
| typedef struct { | ||
| volatile int cur_id; | ||
| unsigned long locks[2]; | ||
| pid_t locks[2]; | ||
| } base_header_t; | ||
|
|
||
| #include "demultiplex.h" | ||
|
|
@@ -43,11 +44,11 @@ typedef struct { | |
| int num_ranks; | ||
| int read_file; | ||
| int num_started; | ||
| unsigned long locks[6]; | ||
| pid_t locks[6]; | ||
| } biter_header_t; | ||
|
|
||
| typedef struct { | ||
| unsigned long locks[2]; | ||
| pid_t locks[2]; | ||
| sheep_ptr_t hash; | ||
| sheep_ptr_t lru_head; | ||
| sheep_ptr_t lru_end; | ||
|
|
||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -347,7 +347,7 @@ realpath_stk (const char *name, char *resolved, struct realpath_bufs *bufs) | |
|
|
||
| /* Length of this file name component; it can be zero if a file | ||
| name ends in '/'. */ | ||
| long startlen = end - start; | ||
| size_t startlen = end - start; | ||
rountree marked this conversation as resolved.
Show resolved
Hide resolved
|
||
|
|
||
| if (startlen == 0) | ||
| break; | ||
|
|
@@ -365,7 +365,7 @@ realpath_stk (const char *name, char *resolved, struct realpath_bufs *bufs) | |
| if (!ISSLASH (dest[-1])) | ||
| *dest++ = '/'; | ||
|
|
||
| while (rname + bufs->rname.length - dest | ||
| while ( (size_t)(rname + bufs->rname.length - dest) | ||
|
Member
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Same as above for not modifying realpath if not necessary |
||
| < startlen + sizeof dir_suffix) | ||
| { | ||
| long dest_offset = dest - rname; | ||
|
|
||
Uh oh!
There was an error while loading. Please reload this page.