[geany/geany-plugins] f369c8: lineoperations: cleaned up var names & unused vars

Branch: refs/heads/master Author: Sylvan Mostert smostert.dev@gmail.com Committer: Sylvan Mostert smostert.dev@gmail.com Date: Tue, 12 Jan 2016 17:19:08 UTC Commit: f369c8421022ac2e53edd003ba32ad48da92048b https://github.com/geany/geany-plugins/commit/f369c8421022ac2e53edd003ba32ad...

Log Message: ----------- lineoperations: cleaned up var names & unused vars

The variable 'numlines' was removed since it was unnecessary.

Variable names and function are changed to have consistent naming. Now they have underscore between each distinct word.

Modified Paths: -------------- lineoperations/src/linefunctions.c

Modified: lineoperations/src/linefunctions.c 294 lines changed, 143 insertions(+), 151 deletions(-) =================================================================== @@ -23,34 +23,34 @@

// isspace() -gboolean issp(gchar c) +gboolean is_sp(gchar c) { return (c == ' ' || c == '\t' || c == '\f' || c == '\v' || c == '\r' || c == '\n'); }

-// iswhitespaceline(gchar* line) checks if line is a whitespace line -gboolean iswhitespaceline(gchar* line) +// is_whitespace_line(gchar* line) checks if line is a whitespace line +gboolean is_whitespace_line(gchar* line) { gint i;

for(i = 0; line[i] != '\0'; i++) - if(!issp(line[i])) + if(!is_sp(line[i])) return FALSE; return TRUE; }

// comparison function to be used in qsort -static gint compareasc(const void * a, const void * b) +static gint compare_asc(const void * a, const void * b) { return strcmp(*(const gchar **) a, *(const gchar **) b); }

// comparison function to be used in qsort -static gint comparedesc(const void * a, const void * b) +static gint compare_desc(const void * a, const void * b) { return strcmp(*(const gchar **) b, *(const gchar **) a); } @@ -58,52 +58,50 @@ static gint comparedesc(const void * a, const void * b)

// Remove Duplicate Lines, sorted void rmdupst(GeanyDocument *doc) { - gint totalnumchars; // number of characters in the document - gint totalnumlines; // number of lines in the document + gint total_num_chars; // number of characters in the document + gint total_num_lines; // number of lines in the document gchar **lines; // array to hold all lines in the document - gint numlines; // number of lines in **lines array - gchar *newfile; // *final* string to replace current document - gint nfposn; // keeps track of the position of newfile string + gchar *new_file; // *final* string to replace current document + gint nfposn; // keeps track of the position of new_file string gint i; // iterator gint j; // iterator gint k; // iterator

- totalnumchars = sci_get_length(doc->editor->sci); - totalnumlines = sci_get_line_count(doc->editor->sci); - lines = g_malloc(sizeof(gchar *) * totalnumlines); - newfile = g_malloc(sizeof(gchar) * (totalnumchars+1)); - numlines = 0; - nfposn = 0; - k = 0; + total_num_chars = sci_get_length(doc->editor->sci); + total_num_lines = sci_get_line_count(doc->editor->sci); + lines = g_malloc(sizeof(gchar *) * total_num_lines); + new_file = g_malloc(sizeof(gchar) * (total_num_chars+1)); + nfposn = 0; + k = 0;

// copy *all* lines into **lines array - for(i = 0; i < totalnumlines; i++) - lines[numlines++] = sci_get_line(doc->editor->sci, i); + for(i = 0; i < total_num_lines; i++) + lines[i] = sci_get_line(doc->editor->sci, i);

- qsort(lines, numlines, sizeof(gchar *), compareasc); + qsort(lines, total_num_lines, sizeof(gchar *), compare_asc);

- if(numlines > 0) // copy the first line into *newfile + if(total_num_lines > 0) // copy the first line into *new_file for(j = 0; lines[0][j] != '\0'; j++) - newfile[nfposn++] = lines[0][j]; + new_file[nfposn++] = lines[0][j];

- for(i = 0; i < numlines; i++) // copy 1st occurance of each line - // into *newfile + for(i = 0; i < total_num_lines; i++) // copy 1st occurance of each line + // into *new_file if(strcmp(lines[k], lines[i]) != 0) { for(j = 0; lines[i][j] != '\0'; j++) - newfile[nfposn++] = lines[i][j]; + new_file[nfposn++] = lines[i][j]; k = i; }

- newfile[nfposn] = '\0'; - sci_set_text(doc->editor->sci, newfile); // set new document + new_file[nfposn] = '\0'; + sci_set_text(doc->editor->sci, new_file); // set new document

// free used memory - for(i = 0; i < numlines; i++) + for(i = 0; i < total_num_lines; i++) g_free(lines[i]); g_free(lines); g_free(new_file); @@ -112,58 +110,56 @@ void rmdupst(GeanyDocument *doc) {

// Remove Duplicate Lines, ordered void rmdupln(GeanyDocument *doc) { - gint totalnumchars; // number of characters in the document - gint totalnumlines; // number of lines in the document + gint total_num_chars; // number of characters in the document + gint total_num_lines; // number of lines in the document gchar **lines; // array to hold all lines in the document - gint numlines; // number of lines in lines - gchar *newfile; // *final* string to replace current document - gint nfposn; // keeps track of the position of newfile string + gchar *new_file; // *final* string to replace current document + gint nfposn; // keeps track of the position of new_file string gint i; // iterator gint j; // iterator - gboolean *toremove; // flag to 'mark' which lines to remove + gboolean *to_remove; // flag to 'mark' which lines to remove

- totalnumchars = sci_get_length(doc->editor->sci); - totalnumlines = sci_get_line_count(doc->editor->sci); - lines = g_malloc(sizeof(gchar *) * totalnumlines); - newfile = g_malloc(sizeof(gchar) * (totalnumchars+1)); - numlines = 0; - nfposn = 0; - toremove = NULL; + total_num_chars = sci_get_length(doc->editor->sci); + total_num_lines = sci_get_line_count(doc->editor->sci); + lines = g_malloc(sizeof(gchar *) * total_num_lines); + new_file = g_malloc(sizeof(gchar) * (total_num_chars+1)); + nfposn = 0; + to_remove = NULL;

// copy *all* lines into **lines array - for(i = 0; i < totalnumlines; i++) - lines[numlines++] = sci_get_line(doc->editor->sci, i); + for(i = 0; i < total_num_lines; i++) + lines[i] = sci_get_line(doc->editor->sci, i);

- // allocate and set *toremove to all FALSE - toremove = g_malloc(sizeof(gboolean) * numlines); - for(i = 0; i < (numlines); i++) - toremove[i] = FALSE; + // allocate and set *to_remove to all FALSE + to_remove = g_malloc(sizeof(gboolean) * total_num_lines); + for(i = 0; i < (total_num_lines); i++) + to_remove[i] = FALSE;

// find which lines are duplicate - for(i = 0; i < numlines; i++) + for(i = 0; i < total_num_lines; i++) // make sure that the line is not already duplicate - if(!toremove[i]) - for(j = (i+1); j < numlines; j++) - if(!toremove[j]) + if(!to_remove[i]) + for(j = (i+1); j < total_num_lines; j++) + if(!to_remove[j]) if(strcmp(lines[i], lines[j]) == 0) { - toremove[j] = TRUE; // this line is duplicate, - // mark to remove - //toremove[i] = TRUE; //remove all occurrances + to_remove[j] = TRUE; // this line is duplicate, + // mark to remove + //to_remove[i] = TRUE; //remove all occurrances }

- // copy line into 'newfile' if it is not FALSE(not duplicate) - for(i = 0; i < numlines; i++) + // copy line into 'new_file' if it is not FALSE(not duplicate) + for(i = 0; i < total_num_lines; i++) { - if(!toremove[i]) + if(!to_remove[i]) for(j = 0; lines[i][j] != '\0'; j++) - newfile[nfposn++] = lines[i][j]; + new_file[nfposn++] = lines[i][j]; g_free(lines[i]); }

- newfile[nfposn] = '\0'; - sci_set_text(doc->editor->sci, newfile); // set new document + new_file[nfposn] = '\0'; + sci_set_text(doc->editor->sci, new_file); // set new document

@@ -176,58 +172,56 @@ void rmdupln(GeanyDocument *doc) {

void rmunqln(GeanyDocument *doc) { - gint totalnumchars; // number of characters in the document - gint totalnumlines; // number of lines in the document - gchar **lines; // array to hold all lines in the document - gint numlines; // number of lines in lines - gchar *newfile; // *final* string to replace current document - gint nfposn; // keeps track of the position of newfile string - gint i; // iterator - gint j; // iterator - gboolean *toremove; // to 'mark' which lines to remove - - totalnumchars = sci_get_length(doc->editor->sci); - totalnumlines = sci_get_line_count(doc->editor->sci); - lines = g_malloc(sizeof(gchar *) * totalnumlines); - newfile = g_malloc(sizeof(gchar) * (totalnumchars+1)); - numlines = 0; - nfposn = 0; - toremove = NULL; + gint total_num_chars; // number of characters in the document + gint total_num_lines; // number of lines in the document + gchar **lines; // array to hold all lines in the document + gchar *new_file; // *final* string to replace current document + gint nfposn; // keeps track of the position of new_file string + gint i; // iterator + gint j; // iterator + gboolean *to_remove; // to 'mark' which lines to remove + + total_num_chars = sci_get_length(doc->editor->sci); + total_num_lines = sci_get_line_count(doc->editor->sci); + lines = g_malloc(sizeof(gchar *) * total_num_lines); + new_file = g_malloc(sizeof(gchar) * (total_num_chars+1)); + nfposn = 0; + to_remove = NULL;

// copy *all* lines into **lines array - for(i = 0; i < totalnumlines; i++) - lines[numlines++] = sci_get_line(doc->editor->sci, i); + for(i = 0; i < total_num_lines; i++) + lines[i] = sci_get_line(doc->editor->sci, i);

- // allocate and set *toremove to all TRUE - toremove = g_malloc(sizeof(gboolean) * numlines); - for(i = 0; i < (numlines); i++) - toremove[i] = TRUE; + // allocate and set *to_remove to all TRUE + to_remove = g_malloc(sizeof(gboolean) * total_num_lines); + for(i = 0; i < (total_num_lines); i++) + to_remove[i] = TRUE;

// set all unique rows to FALSE - // toremove[i] corresponds to lines[i] - for(i = 0; i < numlines; i++) - if(toremove[i]) - for(j = (i+1); j < numlines; j++) - if(toremove[j]) + // to_remove[i] corresponds to lines[i] + for(i = 0; i < total_num_lines; i++) + if(to_remove[i]) + for(j = (i+1); j < total_num_lines; j++) + if(to_remove[j]) if(strcmp(lines[i], lines[j]) == 0) { - toremove[i] = FALSE; - toremove[j] = FALSE; + to_remove[i] = FALSE; + to_remove[j] = FALSE; }

- // copy line into 'newfile' if it is not FALSE(unique) - for(i = 0; i < numlines; i++) + // copy line into 'new_file' if it is not FALSE(unique) + for(i = 0; i < total_num_lines; i++) { - if(!toremove[i]) + if(!to_remove[i]) for(j = 0; lines[i][j] != '\0'; j++) - newfile[nfposn++] = lines[i][j]; + new_file[nfposn++] = lines[i][j]; g_free(lines[i]); }

- newfile[nfposn] = '\0'; - sci_set_text(doc->editor->sci, newfile); // set new document + new_file[nfposn] = '\0'; + sci_set_text(doc->editor->sci, new_file); // set new document

@@ -240,29 +234,29 @@ void rmunqln(GeanyDocument *doc) {

void rmemtyln(GeanyDocument *doc) { - gint totalnumchars; // number of characters in the document - gint totalnumlines; // number of lines in the document + gint total_num_chars; // number of characters in the document + gint total_num_lines; // number of lines in the document gchar *line; // temporary line gint linelen; // length of *line - gchar *newfile; // *final* string to replace current document - gint nfposn; // position to the last character in newfile + gchar *new_file; // *final* string to replace current document + gint nfposn; // position to the last character in new_file gint i; // temporary iterator number gint j; // iterator

- totalnumchars = sci_get_length(doc->editor->sci); - totalnumlines = sci_get_line_count(doc->editor->sci); + total_num_chars = sci_get_length(doc->editor->sci); + total_num_lines = sci_get_line_count(doc->editor->sci); /* * Allocate space for the new document (same amount as open document. * If the document contains lines with only whitespace, * not all of this space will be used.) */ - newfile = g_malloc(sizeof(gchar) * (totalnumchars+1)); - nfposn = 0; + new_file = g_malloc(sizeof(gchar) * (total_num_chars+1)); + nfposn = 0;

- for(i = 0; i < totalnumlines; i++) // loop through opened doc char by char + for(i = 0; i < total_num_lines; i++) // loop through opened doc char by char { linelen = sci_get_line_length(doc->editor->sci, i);

@@ -271,52 +265,52 @@ void rmemtyln(GeanyDocument *doc) { line = sci_get_line(doc->editor->sci, i);

if(line[0] != '\r') - // copy current line into *newfile + // copy current line into *new_file for(j = 0; line[j] != '\0'; j++) - newfile[nfposn++] = line[j]; + new_file[nfposn++] = line[j]; } else if(linelen != 1) { line = sci_get_line(doc->editor->sci, i);

- // copy current line into *newfile + // copy current line into *new_file for(j = 0; line[j] != '\0'; j++) - newfile[nfposn++] = line[j]; + new_file[nfposn++] = line[j]; } } - newfile[nfposn] = '\0'; - sci_set_text(doc->editor->sci, newfile); // set new document + new_file[nfposn] = '\0'; + sci_set_text(doc->editor->sci, new_file); // set new document

- g_free(newfile); + g_free(new_file); }

// Remove Whitespace Lines void rmwhspln(GeanyDocument *doc) { - gint totalnumchars; // number of characters in the document - gint totalnumlines; // number of lines in the document + gint total_num_chars; // number of characters in the document + gint total_num_lines; // number of lines in the document gchar *line; // temporary line gint linelen; // length of *line - gchar *newfile; // *final* string to replace current document - gint nfposn; // position to the last character in newfile + gchar *new_file; // *final* string to replace current document + gint nfposn; // position to the last character in new_file gint i; // temporary iterator number - gint j; // iterator + gint j; // iterator

- totalnumchars = sci_get_length(doc->editor->sci); - totalnumlines = sci_get_line_count(doc->editor->sci); + total_num_chars = sci_get_length(doc->editor->sci); + total_num_lines = sci_get_line_count(doc->editor->sci); /* * Allocate space for the new document (same amount as open document. * If the document contains lines with only whitespace, * not all of this space will be used.) */ - newfile = g_malloc(sizeof(gchar) * (totalnumchars+1)); - nfposn = 0; + new_file = g_malloc(sizeof(gchar) * (total_num_chars+1)); + nfposn = 0;

- for(i = 0; i < totalnumlines; i++) // loop through opened doc + for(i = 0; i < total_num_lines; i++) // loop through opened doc { linelen = sci_get_line_length(doc->editor->sci, i);

@@ -325,68 +319,66 @@ void rmwhspln(GeanyDocument *doc) { line = sci_get_line(doc->editor->sci, i);

if(line[0] != '\r') - // copy current line into *newfile + // copy current line into *new_file for(j = 0; line[j] != '\0'; j++) - newfile[nfposn++] = line[j]; + new_file[nfposn++] = line[j]; } else if(linelen != 1) { line = sci_get_line(doc->editor->sci, i);

- if(!iswhitespaceline(line)) - // copy current line into *newfile + if(!is_whitespace_line(line)) + // copy current line into *new_file for(j = 0; line[j] != '\0'; j++) - newfile[nfposn++] = line[j]; + new_file[nfposn++] = line[j]; }

} - newfile[nfposn] = '\0'; - sci_set_text(doc->editor->sci, newfile); // set new document + new_file[nfposn] = '\0'; + sci_set_text(doc->editor->sci, new_file); // set new document

- g_free(newfile); + g_free(new_file); }

// Sort Lines Ascending and Descending void sortlines(GeanyDocument *doc, gboolean asc) { - gint totalnumchars; // number of characters in the document - gint totalnumlines; // number of lines in the document + gint total_num_chars; // number of characters in the document + gint total_num_lines; // number of lines in the document gchar **lines; // array to hold all lines in the document - gint numlines; // number of lines in **lines array - gchar *newfile; // *final* string to replace current document - gint nfposn; // keeps track of the position of newfile string + gchar *new_file; // *final* string to replace current document + gint nfposn; // keeps track of the position of new_file string gint i; // iterator gint j; // iterator

- totalnumchars = sci_get_length(doc->editor->sci); - totalnumlines = sci_get_line_count(doc->editor->sci); - lines = g_malloc(sizeof(gchar *) * totalnumlines); - newfile = g_malloc(sizeof(gchar) * (totalnumchars+1)); - numlines = 0; - nfposn = 0; + total_num_chars = sci_get_length(doc->editor->sci); + total_num_lines = sci_get_line_count(doc->editor->sci); + lines = g_malloc(sizeof(gchar *) * total_num_lines); + new_file = g_malloc(sizeof(gchar) * (total_num_chars+1)); + nfposn = 0;

// copy *all* lines into **lines array - for(i = 0; i < totalnumlines; i++) - lines[numlines++] = sci_get_line(doc->editor->sci, i); + for(i = 0; i < total_num_lines; i++) + lines[i] = sci_get_line(doc->editor->sci, i);

// sort **lines array if(asc) - qsort(lines, numlines, sizeof(gchar *), compareasc); + qsort(lines, total_num_lines, sizeof(gchar *), compare_asc); else - qsort(lines, numlines, sizeof(gchar *), comparedesc); + qsort(lines, total_num_lines, sizeof(gchar *), compare_desc);

- // copy **lines array into *newfile - for(i = 0; i < numlines; i++) + // copy **lines array into *new_file + for(i = 0; i < total_num_lines; i++) { for(j = 0; lines[i][j] != '\0'; j++) - newfile[nfposn++] = lines[i][j]; + new_file[nfposn++] = lines[i][j];

g_free(lines[i]); } - newfile[nfposn] = '\0'; - sci_set_text(doc->editor->sci, newfile); // set new document + new_file[nfposn] = '\0'; + sci_set_text(doc->editor->sci, new_file); // set new document

-------------- This E-Mail was brought to you by github_commit_mail.py (Source: https://github.com/geany/infrastructure).