Skip to content

Commit a285b77

Browse files
author
luke
committed
Add missing parens in Windows macros and remove defensive ones in context.c.
git-svn-id: https://svn.r-project.org/R/trunk@86205 00db46b3-68df-0310-9c12-caf00c1e9a41
1 parent b7d1dab commit a285b77

File tree

2 files changed

+5
-5
lines changed

2 files changed

+5
-5
lines changed

src/gnuwin32/fixed/h/psignal.h

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -163,14 +163,14 @@ int sigsuspend(sigset_t* sigset_Info);
163163
164164
we only currently use the case sm=0, so avoid compiler warnings by */
165165

166-
#define sigsetjmp(jb, sm) (jb->mask_was_saved=0, setjmp(jb->jmpbuf))
166+
#define sigsetjmp(jb, sm) ((jb)->mask_was_saved=0, setjmp((jb)->jmpbuf))
167167

168168

169169
/* We can transform this in a function but ... */
170170

171-
#define siglongjmp(jb, val) (((jb->mask_was_saved)?\
172-
sigprocmask(SIG_SETMASK, &jb->saved_mask, 0):0),\
173-
longjmp(jb->jmpbuf, val))
171+
#define siglongjmp(jb, val) ((((jb)->mask_was_saved)?\
172+
sigprocmask(SIG_SETMASK, &(jb)->saved_mask, 0):0),\
173+
longjmp((jb)->jmpbuf, val))
174174

175175

176176

src/main/context.c

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -241,7 +241,7 @@ attribute_hidden void NORET R_jumpctxt(RCNTXT * targetcptr, int mask, SEXP val)
241241
/* usually cptr->cjmpbuf_ptr == NULL, but bcEval_loop() sets a
242242
non-NULL value to share a jmpbuf among several contexts */
243243
if (cptr->cjmpbuf_ptr)
244-
LONGJMP((*(cptr->cjmpbuf_ptr)), mask); //**** extra parens until Windows macro is fixed
244+
LONGJMP(*(cptr->cjmpbuf_ptr), mask);
245245
else
246246
LONGJMP(cptr->cjmpbuf, mask);
247247
}

0 commit comments

Comments
 (0)