Revert "glcpp: Rewrite line-continuation support to act globally."
This reverts commit 962a1c07b4
.
Further testing revealed that this commit can cause the pre-processor to enter
infinite loops. For now, simply revert this code until a cleaner,
better-tested version is available.
This commit is contained in:
@@ -70,59 +70,82 @@ glcpp_warning (YYLTYPE *locp, glcpp_parser_t *parser, const char *fmt, ...)
|
|||||||
&parser->info_log_length, "\n");
|
&parser->info_log_length, "\n");
|
||||||
}
|
}
|
||||||
|
|
||||||
/* Remove any line continuation characters in the shader, (whether in
|
/* Searches backwards for '^ *#' from a given starting point. */
|
||||||
* preprocessing directives or in GLSL code).
|
static int
|
||||||
|
in_directive(const char *shader, const char *ptr)
|
||||||
|
{
|
||||||
|
assert(ptr >= shader);
|
||||||
|
|
||||||
|
/* Search backwards for '#'. If we find a \n first, it doesn't count */
|
||||||
|
for (; ptr >= shader && *ptr != '#'; ptr--) {
|
||||||
|
if (*ptr == '\n')
|
||||||
|
return 0;
|
||||||
|
}
|
||||||
|
if (ptr >= shader) {
|
||||||
|
/* Found '#'...look for spaces preceded by a newline */
|
||||||
|
for (ptr--; ptr >= shader && isblank(*ptr); ptr--);
|
||||||
|
// FIXME: I don't think the '\n' case can happen
|
||||||
|
if (ptr < shader || *ptr == '\n')
|
||||||
|
return 1;
|
||||||
|
}
|
||||||
|
return 0;
|
||||||
|
}
|
||||||
|
|
||||||
|
/* Remove any line continuation characters in preprocessing directives.
|
||||||
|
* However, ignore any in GLSL code, as "There is no line continuation
|
||||||
|
* character" (1.30 page 9) in GLSL.
|
||||||
*/
|
*/
|
||||||
static char *
|
static char *
|
||||||
remove_line_continuations(glcpp_parser_t *ctx, const char *shader)
|
remove_line_continuations(glcpp_parser_t *ctx, const char *shader)
|
||||||
{
|
{
|
||||||
|
int in_continued_line = 0;
|
||||||
|
int extra_newlines = 0;
|
||||||
char *clean = ralloc_strdup(ctx, "");
|
char *clean = ralloc_strdup(ctx, "");
|
||||||
const char *backslash, *newline;
|
const char *search_start = shader;
|
||||||
int collapsed_newlines = 0;
|
const char *newline;
|
||||||
|
while ((newline = strchr(search_start, '\n')) != NULL) {
|
||||||
|
const char *backslash = NULL;
|
||||||
|
|
||||||
while (true) {
|
/* # of characters preceding the newline. */
|
||||||
backslash = strchr(shader, '\\');
|
int n = newline - shader;
|
||||||
|
|
||||||
/* If we have previously collapsed any line-continuations,
|
/* Find the preceding '\', if it exists */
|
||||||
* then we want to insert additional newlines at the next
|
if (n >= 1 && newline[-1] == '\\')
|
||||||
* occurrence of a newline character to avoid changing any
|
backslash = newline - 1;
|
||||||
* line numbers.
|
else if (n >= 2 && newline[-1] == '\r' && newline[-2] == '\\')
|
||||||
*/
|
backslash = newline - 2;
|
||||||
if (collapsed_newlines) {
|
|
||||||
newline = strchr(shader, '\n');
|
/* Double backslashes don't count (the backslash is escaped) */
|
||||||
if (newline &&
|
if (backslash != NULL && backslash[-1] == '\\') {
|
||||||
(backslash == NULL || newline < backslash))
|
backslash = NULL;
|
||||||
{
|
}
|
||||||
ralloc_strncat(&clean, shader,
|
|
||||||
newline - shader + 1);
|
if (backslash != NULL) {
|
||||||
while (collapsed_newlines--)
|
/* We found a line continuation, but do we care? */
|
||||||
ralloc_strcat(&clean, "\n");
|
if (!in_continued_line) {
|
||||||
shader = newline + 1;
|
if (in_directive(shader, backslash)) {
|
||||||
|
in_continued_line = 1;
|
||||||
|
extra_newlines = 0;
|
||||||
|
}
|
||||||
}
|
}
|
||||||
|
if (in_continued_line) {
|
||||||
|
/* Copy everything before the \ */
|
||||||
|
ralloc_strncat(&clean, shader, backslash - shader);
|
||||||
|
shader = newline + 1;
|
||||||
|
extra_newlines++;
|
||||||
|
}
|
||||||
|
} else if (in_continued_line) {
|
||||||
|
/* Copy everything up to and including the \n */
|
||||||
|
ralloc_strncat(&clean, shader, newline - shader + 1);
|
||||||
|
shader = newline + 1;
|
||||||
|
/* Output extra newlines to make line numbers match */
|
||||||
|
for (; extra_newlines > 0; extra_newlines--)
|
||||||
|
ralloc_strcat(&clean, "\n");
|
||||||
|
in_continued_line = 0;
|
||||||
}
|
}
|
||||||
|
search_start = newline + 1;
|
||||||
if (backslash == NULL)
|
|
||||||
break;
|
|
||||||
|
|
||||||
/* At each line continuation, (backslash followed by a
|
|
||||||
* newline), copy all preceding text to the output, then
|
|
||||||
* advance the shader pointer to the character after the
|
|
||||||
* newline.
|
|
||||||
*/
|
|
||||||
if (backslash[1] == '\n' ||
|
|
||||||
(backslash[1] == '\r' && backslash[2] == '\n'))
|
|
||||||
{
|
|
||||||
collapsed_newlines++;
|
|
||||||
ralloc_strncat(&clean, shader, backslash - shader);
|
|
||||||
if (backslash[1] == '\n')
|
|
||||||
shader = backslash + 2;
|
|
||||||
else
|
|
||||||
shader = backslash + 3;
|
|
||||||
}
|
|
||||||
}
|
}
|
||||||
|
|
||||||
ralloc_strcat(&clean, shader);
|
ralloc_strcat(&clean, shader);
|
||||||
|
|
||||||
return clean;
|
return clean;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
Reference in New Issue
Block a user