Plugins-Commits July 2012

plugins-commits@lists.geany.org

1 participants
220 discussions

[geany/geany-plugins] b38189: PrettyPrinter: Don't use C++-style comments
by Colomban Wendling 08 Jul '12

08 Jul '12

Branch: refs/heads/1.22_release Author: Colomban Wendling <ban(a)herbesfolles.org> Committer: Colomban Wendling <ban(a)herbesfolles.org> Date: Sun, 08 Jul 2012 21:10:53 Commit: b38189f5380a84cc2ae437254dd78ec4dae839ba https://github.com/geany/geany-plugins/commit/b38189f5380a84cc2ae437254dd78… Log Message: ----------- PrettyPrinter: Don't use C++-style comments Modified Paths: -------------- pretty-printer/src/ConfigUI.c pretty-printer/src/ConfigUI.h pretty-printer/src/PluginEntry.c pretty-printer/src/PluginEntry.h pretty-printer/src/PrettyPrinter.c pretty-printer/src/PrettyPrinter.h Modified: pretty-printer/src/ConfigUI.c 16 files changed, 8 insertions(+), 8 deletions(-) =================================================================== @@ -18,15 +18,15 @@ #include "ConfigUI.h" -//======================= FUNCTIONS ==================================================================== +/*======================= FUNCTIONS ====================================================================*/ -//static GtkWidget* createTwoOptionsBox(const char* label, const char* checkBox1, const char* checkBox2, gboolean cb1Active, gboolean cb2Active, GtkWidget** option1, GtkWidget** option2); +/*static GtkWidget* createTwoOptionsBox(const char* label, const char* checkBox1, const char* checkBox2, gboolean cb1Active, gboolean cb2Active, GtkWidget** option1, GtkWidget** option2);*/ static GtkWidget* createThreeOptionsBox(const char* label, const char* checkBox1, const char* checkBox2, const char* checkBox3, gboolean cb1Active, gboolean cb2Active, gboolean cb3Active, GtkWidget** option1, GtkWidget** option2, GtkWidget** option3); static GtkWidget* createEmptyTextOptions(gboolean emptyNodeStripping, gboolean emptyNodeStrippingSpace, gboolean forceEmptyNodeSplit); static GtkWidget* createIndentationOptions(char indentation, int count); static GtkWidget* createLineReturnOptions(const char* lineReturn); -//============================================ PRIVATE PROPERTIES ====================================== +/*============================================ PRIVATE PROPERTIES ======================================*/ static GtkWidget* commentOneLine; static GtkWidget* commentInline; @@ -44,15 +44,15 @@ static GtkWidget* indentationCount; static GtkWidget* lineBreak; -//============================================= PUBLIC FUNCTIONS ======================================== +/*============================================= PUBLIC FUNCTIONS ========================================*/ -//redeclaration of extern variable +/* redeclaration of extern variable */ PrettyPrintingOptions* prettyPrintingOptions; -//Will never be used, just here for example +/* Will never be used, just here for example */ GtkWidget* createPrettyPrinterConfigUI(GtkDialog * dialog) { - //default printing options + /* default printing options */ if (prettyPrintingOptions == NULL) { prettyPrintingOptions = createDefaultPrettyPrintingOptions(); } PrettyPrintingOptions* ppo = prettyPrintingOptions; @@ -111,7 +111,7 @@ void saveSettings(void) else ppo->newLineChars = "\r\n"; } -//============================================= PRIVATE FUNCTIONS ======================================= +/*============================================= PRIVATE FUNCTIONS =======================================*/ /*GtkWidget* createTwoOptionsBox(const char* label, const char* checkBox1, Modified: pretty-printer/src/ConfigUI.h 6 files changed, 3 insertions(+), 3 deletions(-) =================================================================== @@ -19,7 +19,7 @@ #ifndef PP_CONFIG_UI_H #define PP_CONFIG_UI_H -//========================================== INCLUDES ========================================================= +/*========================================== INCLUDES =========================================================*/ #include <gtk/gtk.h> @@ -30,11 +30,11 @@ #include "PrettyPrinter.h" -//========================================== VARIABLES ======================================================== +/*========================================== VARIABLES ========================================================*/ extern PrettyPrintingOptions* prettyPrintingOptions; -//========================================== FUNCTIONS ======================================================== +/*========================================== FUNCTIONS ========================================================*/ GtkWidget* createPrettyPrinterConfigUI(GtkDialog* dialog); void saveSettings(void); Modified: pretty-printer/src/PluginEntry.c 56 files changed, 28 insertions(+), 28 deletions(-) =================================================================== @@ -28,7 +28,7 @@ GeanyData* geany_data; GeanyFunctions* geany_functions; -//========================================== PLUGIN INFORMATION ========================================================== +/*========================================== PLUGIN INFORMATION ==========================================================*/ PLUGIN_VERSION_CHECK(130) PLUGIN_SET_TRANSLATABLE_INFO( @@ -39,11 +39,11 @@ PRETTY_PRINTER_VERSION, "Cédric Tabin - http://www.astorm.ch") PLUGIN_KEY_GROUP(prettyprinter, 1) -//========================================== DECLARATIONS ================================================================ +/*========================================== DECLARATIONS ================================================================*/ -static GtkWidget* main_menu_item = NULL; //the main menu of the plugin +static GtkWidget* main_menu_item = NULL; /*the main menu of the plugin*/ -//declaration of the functions +/* declaration of the functions */ static void xml_format(GtkMenuItem *menuitem, gpointer gdata); static void kb_run_xml_pretty_print(G_GNUC_UNUSED guint key_id); static void config_closed(GtkWidget* configWidget, gint response, gpointer data); @@ -52,51 +52,51 @@ void plugin_cleanup(void); GtkWidget* plugin_configure(GtkDialog * dialog); -//========================================== FUNCTIONS =================================================================== +/*========================================== FUNCTIONS ===================================================================*/ void plugin_init(GeanyData *data) { - //initializes the libxml2 + /* initializes the libxml2 */ LIBXML_TEST_VERSION - //mutilanguage support + /* mutilanguage support */ main_locale_init(LOCALEDIR, GETTEXT_PACKAGE); - //put the menu into the Tools + /* put the menu into the Tools */ main_menu_item = gtk_menu_item_new_with_mnemonic(_("PrettyPrinter XML")); ui_add_document_sensitive(main_menu_item); gtk_widget_show(main_menu_item); gtk_container_add(GTK_CONTAINER(geany->main_widgets->tools_menu), main_menu_item); - //init keybindings + /* init keybindings */ keybindings_set_item(plugin_key_group, 0, kb_run_xml_pretty_print, 0, 0, "run_pretty_printer_xml", _("Run the PrettyPrinter XML"), main_menu_item); - //add activation callback + /* add activation callback */ g_signal_connect(main_menu_item, "activate", G_CALLBACK(xml_format), NULL); } void plugin_cleanup(void) { - //destroys the plugin + /* destroys the plugin */ gtk_widget_destroy(main_menu_item); } GtkWidget* plugin_configure(GtkDialog * dialog) { - //creates the configuration widget + /* creates the configuration widget */ GtkWidget* widget = createPrettyPrinterConfigUI(dialog); g_signal_connect(dialog, "response", G_CALLBACK(config_closed), NULL); return widget; } -//========================================== LISTENERS =================================================================== +/*========================================== LISTENERS ===================================================================*/ void config_closed(GtkWidget* configWidget, gint response, gpointer gdata) { - //if the user clicked OK or APPLY, then save the settings + /* if the user clicked OK or APPLY, then save the settings */ if (response == GTK_RESPONSE_OK || response == GTK_RESPONSE_APPLY) { @@ -111,39 +111,39 @@ void kb_run_xml_pretty_print(G_GNUC_UNUSED guint key_id) void xml_format(GtkMenuItem* menuitem, gpointer gdata) { - //retrieves the current document + /* retrieves the current document */ GeanyDocument* doc = document_get_current(); g_return_if_fail(doc != NULL); GeanyEditor* editor = doc->editor; ScintillaObject* sco = editor->sci; - //default printing options + /* default printing options */ if (prettyPrintingOptions == NULL) { prettyPrintingOptions = createDefaultPrettyPrintingOptions(); } - //prepare the buffer that will contain the text - //from the scintilla object + /* prepare the buffer that will contain the text + * from the scintilla object */ int length = sci_get_length(sco)+1; char* buffer = (char*)malloc(length*sizeof(char)); - if (buffer == NULL) { exit(-1); } //malloc error + if (buffer == NULL) { exit(-1); } /* malloc error */ - //retrieves the text + /* retrieves the text */ sci_get_text(sco, length, buffer); - //checks if the data is an XML format + /* checks if the data is an XML format */ xmlDoc* parsedDocument = xmlParseDoc((unsigned char*)buffer); - //this is not a valid xml => exit with an error message + /* this is not a valid xml => exit with an error message */ if(parsedDocument == NULL) { dialogs_show_msgbox(GTK_MESSAGE_ERROR, _("Unable to parse the content as XML.")); return; } - //free all + /* free all */ xmlFreeDoc(parsedDocument); - //process pretty-printing + /* process pretty-printing */ int result = processXMLPrettyPrinting(&buffer, &length, prettyPrintingOptions); if (result != PRETTY_PRINTING_SUCCESS) { @@ -151,14 +151,14 @@ void xml_format(GtkMenuItem* menuitem, gpointer gdata) return; } - //updates the document + /* updates the document */ sci_set_text(sco, buffer); - //set the line + /* set the line */ int xOffset = scintilla_send_message(sco, SCI_GETXOFFSET, 0, 0); - scintilla_send_message(sco, SCI_LINESCROLL, -xOffset, 0); //TODO update with the right function-call for geany-0.19 + scintilla_send_message(sco, SCI_LINESCROLL, -xOffset, 0); /* TODO update with the right function-call for geany-0.19 */ - //sets the type + /* sets the type */ GeanyFiletype* fileType = filetypes_index(GEANY_FILETYPES_XML); document_set_filetype(doc, fileType); } Modified: pretty-printer/src/PluginEntry.h 6 files changed, 3 insertions(+), 3 deletions(-) =================================================================== @@ -19,7 +19,7 @@ #ifndef PLUGIN_ENTRY_H #define PLUGIN_ENTRY_H -//========================================== INCLUDES ========================================================== +/*========================================== INCLUDES ==========================================================*/ #include <stdlib.h> #include <stdio.h> @@ -34,8 +34,8 @@ #include "PrettyPrinter.h" #include "ConfigUI.h" -//========================================== PROPERTIES ======================================================== +/*========================================== PROPERTIES ========================================================*/ -//========================================== FUNCTIONS ========================================================= +/*========================================== FUNCTIONS =========================================================*/ #endif Modified: pretty-printer/src/PrettyPrinter.c 458 files changed, 229 insertions(+), 229 deletions(-) =================================================================== @@ -18,63 +18,63 @@ #include "PrettyPrinter.h" -//======================= FUNCTIONS ==================================================================== +/*======================= FUNCTIONS ====================================================================*/ -//error reporting functions -static void PP_ERROR(const char* fmt, ...) G_GNUC_PRINTF(1,2); //prints an error message +/* error reporting functions */ +static void PP_ERROR(const char* fmt, ...) G_GNUC_PRINTF(1,2); /* prints an error message */ -//xml pretty printing functions -static void putCharInBuffer(char charToAdd); //put a char into the new char buffer -static void putCharsInBuffer(const char* charsToAdd); //put the chars into the new char buffer -static void putNextCharsInBuffer(int nbChars); //put the next nbChars of the input buffer into the new buffer -static int readWhites(bool considerLineBreakAsWhite); //read the next whites into the input buffer -static char readNextChar(void); //read the next char into the input buffer; -static char getNextChar(void); //returns the next char but do not increase the input buffer index (use readNextChar for that) -static char getPreviousInsertedChar(void); //returns the last inserted char into the new buffer -static bool isWhite(char c); //check if the specified char is a white -static bool isSpace(char c); //check if the specified char is a space -static bool isLineBreak(char c); //check if the specified char is a new line -static bool isQuote(char c); //check if the specified char is a quote (simple or double) -static int putNewLine(void); //put a new line into the new char buffer with the correct number of whites (indentation) -static bool isInlineNodeAllowed(void); //check if it is possible to have an inline node -static bool isOnSingleLine(int skip, char stop1, char stop2); //check if the current node data is on one line (for inlining) -static void resetBackwardIndentation(bool resetLineBreak); //reset the indentation for the current depth (just reset the index in fact) +/* xml pretty printing functions */ +static void putCharInBuffer(char charToAdd); /* put a char into the new char buffer */ +static void putCharsInBuffer(const char* charsToAdd); /* put the chars into the new char buffer */ +static void putNextCharsInBuffer(int nbChars); /* put the next nbChars of the input buffer into the new buffer */ +static int readWhites(bool considerLineBreakAsWhite); /* read the next whites into the input buffer */ +static char readNextChar(void); /* read the next char into the input buffer; */ +static char getNextChar(void); /* returns the next char but do not increase the input buffer index (use readNextChar for that) */ +static char getPreviousInsertedChar(void); /* returns the last inserted char into the new buffer */ +static bool isWhite(char c); /* check if the specified char is a white */ +static bool isSpace(char c); /* check if the specified char is a space */ +static bool isLineBreak(char c); /* check if the specified char is a new line */ +static bool isQuote(char c); /* check if the specified char is a quote (simple or double) */ +static int putNewLine(void); /* put a new line into the new char buffer with the correct number of whites (indentation) */ +static bool isInlineNodeAllowed(void); /* check if it is possible to have an inline node */ +static bool isOnSingleLine(int skip, char stop1, char stop2); /* check if the current node data is on one line (for inlining) */ +static void resetBackwardIndentation(bool resetLineBreak); /* reset the indentation for the current depth (just reset the index in fact) */ -//specific parsing functions -static int processElements(void); //returns the number of elements processed -static void processElementAttribute(void); //process on attribute of a node -static void processElementAttributes(void); //process all the attributes of a node -static void processHeader(void); //process the header <?xml version="..." ?> -static void processNode(void); //process an XML node -static void processTextNode(void); //process a text node -static void processComment(void); //process a comment -static void processCDATA(void); //process a CDATA node -static void processDoctype(void); //process a DOCTYPE node -static void processDoctypeElement(void); //process a DOCTYPE ELEMENT node +/* specific parsing functions */ +static int processElements(void); /* returns the number of elements processed */ +static void processElementAttribute(void); /* process on attribute of a node */ +static void processElementAttributes(void); /* process all the attributes of a node */ +static void processHeader(void); /* process the header <?xml version="..." ?> */ +static void processNode(void); /* process an XML node */ +static void processTextNode(void); /* process a text node */ +static void processComment(void); /* process a comment */ +static void processCDATA(void); /* process a CDATA node */ +static void processDoctype(void); /* process a DOCTYPE node */ +static void processDoctypeElement(void); /* process a DOCTYPE ELEMENT node */ -//debug function -static void printError(const char *msg, ...) G_GNUC_PRINTF(1,2); //just print a message like the printf method -static void printDebugStatus(void); //just print some variables into the console for debugging +/* debug function */ +static void printError(const char *msg, ...) G_GNUC_PRINTF(1,2); /* just print a message like the printf method */ +static void printDebugStatus(void); /* just print some variables into the console for debugging */ -//============================================ PRIVATE PROPERTIES ====================================== +/*============================================ PRIVATE PROPERTIES ======================================*/ -//those are variables that are shared by the functions and -//shouldn't be altered. +/* those are variables that are shared by the functions and + * shouldn't be altered. */ -static int result; //result of the pretty printing -static char* xmlPrettyPrinted; //new buffer for the formatted XML -static int xmlPrettyPrintedLength; //buffer size -static int xmlPrettyPrintedIndex; //buffer index (position of the next char to insert) -static char* inputBuffer; //input buffer -static int inputBufferLength; //input buffer size -static int inputBufferIndex; //input buffer index (position of the next char to read into the input string) -static int currentDepth; //current depth (for indentation) -static char* currentNodeName; //current node name -static bool appendIndentation; //if the indentation must be added (with a line break before) -static bool lastNodeOpen; //defines if the last action was a not opening or not -static PrettyPrintingOptions* options; //options of PrettyPrinting +static int result; /* result of the pretty printing */ +static char* xmlPrettyPrinted; /* new buffer for the formatted XML */ +static int xmlPrettyPrintedLength; /* buffer size */ +static int xmlPrettyPrintedIndex; /* buffer index (position of the next char to insert) */ +static char* inputBuffer; /* input buffer */ +static int inputBufferLength; /* input buffer size */ +static int inputBufferIndex; /* input buffer index (position of the next char to read into the input string) */ +static int currentDepth; /* current depth (for indentation) */ +static char* currentNodeName; /* current node name */ +static bool appendIndentation; /* if the indentation must be added (with a line break before) */ +static bool lastNodeOpen; /* defines if the last action was a not opening or not */ +static PrettyPrintingOptions* options; /* options of PrettyPrinting */ -//============================================ GENERAL FUNCTIONS ======================================= +/*============================================ GENERAL FUNCTIONS =======================================*/ static void PP_ERROR(const char* fmt, ...) { @@ -88,11 +88,11 @@ static void PP_ERROR(const char* fmt, ...) int processXMLPrettyPrinting(char** buffer, int* length, PrettyPrintingOptions* ppOptions) { - //empty buffer, nothing to process + /* empty buffer, nothing to process */ if (*length == 0) { return PRETTY_PRINTING_EMPTY_XML; } if (buffer == NULL || *buffer == NULL) { return PRETTY_PRINTING_EMPTY_XML; } - //initialize the variables + /* initialize the variables */ result = PRETTY_PRINTING_SUCCESS; bool freeOptions = FALSE; if (ppOptions == NULL) @@ -116,43 +116,43 @@ int processXMLPrettyPrinting(char** buffer, int* length, PrettyPrintingOptions* xmlPrettyPrinted = (char*)malloc(sizeof(char)*(*length)); if (xmlPrettyPrinted == NULL) { PP_ERROR("Allocation error (initialisation)"); return PRETTY_PRINTING_SYSTEM_ERROR; } - //go to the first char + /* go to the first char */ readWhites(TRUE); - //process the pretty-printing + /* process the pretty-printing */ processElements(); - //close the buffer + /* close the buffer */ putCharInBuffer('\0'); - //adjust the final size + /* adjust the final size */ char* reallocated = (char*)realloc(xmlPrettyPrinted, xmlPrettyPrintedIndex); if (reallocated == NULL) { PP_ERROR("Allocation error (reallocation size is %d)", xmlPrettyPrintedIndex); return PRETTY_PRINTING_SYSTEM_ERROR; } xmlPrettyPrinted = reallocated; - //freeing the unused values + /* freeing the unused values */ if (freeOptions) { free(options); } - //if success, then update the values + /* if success, then update the values */ if (result == PRETTY_PRINTING_SUCCESS) { free(*buffer); *buffer = xmlPrettyPrinted; - *length = xmlPrettyPrintedIndex-2; //the '\0' is not in the length + *length = xmlPrettyPrintedIndex-2; /* the '\0' is not in the length */ } - //else clean the other values + /* else clean the other values */ else { free(xmlPrettyPrinted); } - //updating the pointers for the using into the caller function - xmlPrettyPrinted = NULL; //avoid reference - inputBuffer = NULL; //avoid reference - currentNodeName = NULL; //avoid reference - options = NULL; //avoid reference + /* updating the pointers for the using into the caller function */ + xmlPrettyPrinted = NULL; /* avoid reference */ + inputBuffer = NULL; /* avoid reference */ + currentNodeName = NULL; /* avoid reference */ + options = NULL; /* avoid reference */ - //and finally the result + /* and finally the result */ return result; } @@ -198,7 +198,7 @@ void putNextCharsInBuffer(int nbChars) void putCharInBuffer(char charToAdd) { - //check if the buffer is full and reallocation if needed + /* check if the buffer is full and reallocation if needed */ if (xmlPrettyPrintedIndex >= xmlPrettyPrintedLength) { if (charToAdd == '\0') { ++xmlPrettyPrintedLength; } @@ -208,7 +208,7 @@ void putCharInBuffer(char charToAdd) xmlPrettyPrinted = reallocated; } - //putting the char and increase the index for the next one + /* putting the char and increase the index for the next one */ xmlPrettyPrinted[xmlPrettyPrintedIndex] = charToAdd; ++xmlPrettyPrintedIndex; } @@ -291,70 +291,70 @@ bool isLineBreak(char c) bool isInlineNodeAllowed(void) { - //the last action was not an opening => inline not allowed + /* the last action was not an opening => inline not allowed */ if (!lastNodeOpen) { return FALSE; } - int firstChar = getNextChar(); //should be '<' or we are in a text node - int secondChar = inputBuffer[inputBufferIndex+1]; //should be '!' - int thirdChar = inputBuffer[inputBufferIndex+2]; //should be '-' or '[' + int firstChar = getNextChar(); /* should be '<' or we are in a text node */ + int secondChar = inputBuffer[inputBufferIndex+1]; /* should be '!' */ + int thirdChar = inputBuffer[inputBufferIndex+2]; /* should be '-' or '[' */ - //loop through the content up to the next opening/closing node - int currentIndex = inputBufferIndex+1; + /* loop through the content up to the next opening/closing node */ + currentIndex = inputBufferIndex+1; if (firstChar == '<') { - //another node is being open ==> no inline ! + /* another node is being open ==> no inline ! */ if (secondChar != '!') { return FALSE; } - //okay we are in a comment/cdata node, so read until it is closed + /* okay we are in a comment/cdata node, so read until it is closed */ - //select the closing char + /* select the closing char */ char closingComment = '-'; if (thirdChar == '[') { closingComment = ']'; } - //read until closing + /* read until closing */ char oldChar = ' '; - currentIndex += 3; //that bypass meanless chars + currentIndex += 3; /* that bypass meanless chars */ bool loop = TRUE; while (loop) { char current = inputBuffer[currentIndex]; - if (current == closingComment && oldChar == closingComment) { loop = FALSE; } //end of comment/cdata + if (current == closingComment && oldChar == closingComment) { loop = FALSE; } /* end of comment/cdata */ oldChar = current; ++currentIndex; } - //okay now avoid blanks - // inputBuffer[index] is now '>' + /* okay now avoid blanks */ + /* inputBuffer[index] is now '>' */ ++currentIndex; while (isWhite(inputBuffer[currentIndex])) { ++currentIndex; } } else { - //this is a text node. Simply loop to the next '<' + /* this is a text node. Simply loop to the next '<' */ while (inputBuffer[currentIndex] != '<') { ++currentIndex; } } - //check what do we have now - char currentChar = inputBuffer[currentIndex]; + /* check what do we have now */ + currentChar = inputBuffer[currentIndex]; if (currentChar == '<') { - //check if that is a closing node + /* check if that is a closing node */ currentChar = inputBuffer[currentIndex+1]; if (currentChar == '/') { - //as we are in a correct XML (so far...), if the node is - //being directly closed, the inline is allowed !!! + /* as we are in a correct XML (so far...), if the node is */ + /* being directly closed, the inline is allowed !!! */ return TRUE; } } - //inline not allowed... + /* inline not allowed... */ return FALSE; } bool isOnSingleLine(int skip, char stop1, char stop2) { - int currentIndex = inputBufferIndex+skip; //skip the n first chars (in comment ' for the alignment with '' for the alignment with '' + xmlPrettyPrintedIndex -= 7; /* remove indentation spaces */ + putCharsInBuffer("--"); /* reset the first chars of '-->' */ } } } - else if (!options->oneLineComment && !inlineAllowed) //oh ! there is a line break + else if (!options->oneLineComment && !inlineAllowed) /* oh ! there is a line break */ { - //if the comments need to be aligned, just add 5 spaces + /* if the comments need to be aligned, just add 5 spaces */ if (options->alignComment) { - int read = readWhites(FALSE); //strip the whites and new line - if (nextChar == '\r' && read == 0 && getNextChar() == '\n') //handles the \r\n return line + int read = readWhites(FALSE); /* strip the whites and new line */ + if (nextChar == '\r' && read == 0 && getNextChar() == '\n') /* handles the \r\n return line */ { readNextChar(); readWhites(FALSE); } - putNewLine(); //put a new indentation line - putCharsInBuffer(" "); //align with ' + xmlPrettyPrintedIndex -= 11; /* remove indentation spaces */ + putCharsInBuffer("]]"); /* reset the first chars of '-->' */ } } } - else if (!options->oneLineCdata && !inlineAllowed) //line break + else if (!options->oneLineCdata && !inlineAllowed) /* line break */ { - //if the cdata need to be aligned, just add 9 spaces + /* if the cdata need to be aligned, just add 9 spaces */ if (options->alignCdata) { - int read = readWhites(FALSE); //strip the whites and new line - if (nextChar == '\r' && read == 0 && getNextChar() == '\n') //handles the \r\n return line + int read = readWhites(FALSE); /* strip the whites and new line */ + if (nextChar == '\r' && read == 0 && getNextChar() == '\n') /* handles the \r\n return line */ { readNextChar(); readWhites(FALSE); } - putNewLine(); //put a new indentation line - putCharsInBuffer(" "); //align with <![CDATA[ - oldChar = ' '; //and update the last char + putNewLine(); /* put a new indentation line */ + putCharsInBuffer(" "); /* align with <![CDATA[ */ + oldChar = ' '; /* and update the last char */ } else { @@ -984,11 +984,11 @@ void processCDATA(void) oldChar = nextChar; } } - else //cdata are inlined + else /* cdata are inlined */ { - readWhites(TRUE); //strip the whites and add a space if necessary + readWhites(TRUE); /* strip the whites and add a space if necessary */ if(getPreviousInsertedChar() != ' ' && - strncmp(xmlPrettyPrinted+xmlPrettyPrintedIndex-9, "<![CDATA[", 9) != 0) //prevents adding a space at the beginning + strncmp(xmlPrettyPrinted+xmlPrettyPrintedIndex-9, "<![CDATA[", 9) != 0) /* prevents adding a space at the beginning */ { putCharInBuffer(' '); oldChar = ' '; @@ -996,10 +996,10 @@ void processCDATA(void) } } - //if the cdata is inline, then all the trailing spaces are removed + /* if the cdata is inline, then all the trailing spaces are removed */ if (options->oneLineCdata) { - xmlPrettyPrintedIndex -= 2; //because of the last ']]' inserted + xmlPrettyPrintedIndex -= 2; /* because of the last ']]' inserted */ while(isWhite(xmlPrettyPrinted[xmlPrettyPrintedIndex-1])) { --xmlPrettyPrintedIndex; @@ -1007,8 +1007,8 @@ void processCDATA(void) putCharsInBuffer("]]"); } - //finalize the cdata - char lastChar = readNextChar(); //should be '>' + /* finalize the cdata */ + lastChar = readNextChar(); /* should be '>' */ if (lastChar != '>') { printError("processCDATA : last char must be '>' (not '%c')", lastChar); @@ -1020,38 +1020,38 @@ void processCDATA(void) if (inlineAllowed) { appendIndentation = FALSE; } - //there was no node open + /* there was no node open */ lastNodeOpen = FALSE; } void processDoctype(void) { - putNextCharsInBuffer(9); //put the '<!DOCTYPE' into the buffer + putNextCharsInBuffer(9); /* put the '<!DOCTYPE' into the buffer */ bool loop = TRUE; while(loop) { readWhites(TRUE); - putCharInBuffer(' '); //only one space for the attributes + putCharInBuffer(' '); /* only one space for the attributes */ int nextChar = readNextChar(); while(!isWhite(nextChar) && - !isQuote(nextChar) && //begins a quoted text - nextChar != '=' && //begins an attribute - nextChar != '>' && //end of doctype - nextChar != '[') //inner <!ELEMENT> types + !isQuote(nextChar) && /* begins a quoted text */ + nextChar != '=' && /* begins an attribute */ + nextChar != '>' && /* end of doctype */ + nextChar != '[') /* inner <!ELEMENT> types */ { putCharInBuffer(nextChar); nextChar = readNextChar(); } - if (isWhite(nextChar)) {} //do nothing, just let the next loop do the job + if (isWhite(nextChar)) {} /* do nothing, just let the next loop do the job */ else if (isQuote(nextChar) || nextChar == '=') { if (nextChar == '=') { putCharInBuffer(nextChar); - nextChar = readNextChar(); //now we should have a quote + nextChar = readNextChar(); /* now we should have a quote */ if (!isQuote(nextChar)) { @@ -1061,7 +1061,7 @@ void processDoctype(void) } } - //simply process the content + /* simply process the content */ char quote = nextChar; do { @@ -1069,14 +1069,14 @@ void processDoctype(void) nextChar = readNextChar(); } while (nextChar != quote); - putCharInBuffer(nextChar); //now the last char is the last quote + putCharInBuffer(nextChar); /* now the last char is the last quote */ } - else if (nextChar == '>') //end of doctype + else if (nextChar == '>') /* end of doctype */ { putCharInBuffer(nextChar); loop = FALSE; } - else //the char is a '[' => not supported yet + else /* the char is a '[' => not supported yet */ { printError("DOCTYPE inner ELEMENT is currently not supported by PrettyPrinter\n"); result = PRETTY_PRINTING_NOT_SUPPORTED_YET; Modified: pretty-printer/src/PrettyPrinter.h 46 files changed, 23 insertions(+), 23 deletions(-) =================================================================== @@ -19,7 +19,7 @@ #ifndef PRETTY_PRINTER_H #define PRETTY_PRINTER_H -//========================================== INCLUDES ========================================================== +/*========================================== INCLUDES ==========================================================*/ #ifdef HAVE_CONFIG_H # include "config.h" @@ -34,7 +34,7 @@ #include <glib.h> #endif -//========================================== DEFINES =========================================================== +/*========================================== DEFINES ===========================================================*/ #define PRETTY_PRINTER_VERSION "1.3" @@ -54,7 +54,7 @@ typedef unsigned int bool; -//========================================== STRUCTURES ======================================================= +/*========================================== STRUCTURES =======================================================*/ /** * The PrettyPrintingOptions struct allows the programmer to tell the @@ -62,29 +62,29 @@ */ typedef struct { - const char* newLineChars; //char used to generate a new line (generally \r\n) - char indentChar; //char used for indentation - int indentLength; //number of char to use for indentation (by default 2 spaces) - bool oneLineText; //text is put on one line - bool inlineText; //if possible text are inline (no return after the opening node and before closing node) - bool oneLineComment; //comments are put on one line - bool inlineComment; //if possible comments are inline (no return after the opening node and before closing node) - bool oneLineCdata; //cdata are put on one line - bool inlineCdata; //if possible cdata are inline (no return after the opening node and before closing node) - bool emptyNodeStripping; //the empty nodes such <node></node> are set to <node/> - bool emptyNodeStrippingSpace; //put a space before the '/>' when a node is stripped - bool forceEmptyNodeSplit; //force an empty node to be splitted : <node /> becomes <node></node> (only if emptyNodeStripping = false) - bool trimLeadingWhites; //trim the leading whites in a text node - bool trimTrailingWhites; //trim the trailing whites in a text node - bool alignComment; //align the comments. If false, comments are untouched (only if oneLineComment = false) - bool alignText; //align the text in a node. If false, text is untouched (only if oneLineText = false) - bool alignCdata; //align the cdata. If false, cdata is untouched (only if oneLineCdata = false) + const char* newLineChars; /* char used to generate a new line (generally \r\n) */ + char indentChar; /* char used for indentation */ + int indentLength; /* number of char to use for indentation (by default 2 spaces) */ + bool oneLineText; /* text is put on one line */ + bool inlineText; /* if possible text are inline (no return after the opening node and before closing node) */ + bool oneLineComment; /* comments are put on one line */ + bool inlineComment; /* if possible comments are inline (no return after the opening node and before closing node) */ + bool oneLineCdata; /* cdata are put on one line */ + bool inlineCdata; /* if possible cdata are inline (no return after the opening node and before closing node) */ + bool emptyNodeStripping; /* the empty nodes such <node></node> are set to <node/> */ + bool emptyNodeStrippingSpace; /* put a space before the '/>' when a node is stripped */ + bool forceEmptyNodeSplit; /* force an empty node to be splitted : <node /> becomes <node></node> (only if emptyNodeStripping = false) */ + bool trimLeadingWhites; /* trim the leading whites in a text node */ + bool trimTrailingWhites; /* trim the trailing whites in a text node */ + bool alignComment; /* align the comments. If false, comments are untouched (only if oneLineComment = false) */ + bool alignText; /* align the text in a node. If false, text is untouched (only if oneLineText = false) */ + bool alignCdata; /* align the cdata. If false, cdata is untouched (only if oneLineCdata = false) */ } PrettyPrintingOptions; -//========================================== FUNCTIONS ========================================================= +/*========================================== FUNCTIONS =========================================================*/ -int processXMLPrettyPrinting(char** xml, int* length, PrettyPrintingOptions* ppOptions); //process the pretty-printing on a valid xml string (no check done !!!). The ppOptions ARE NOT FREE-ED after processing. The method returns 0 if the pretty-printing has been done. -PrettyPrintingOptions* createDefaultPrettyPrintingOptions(void); //creates a default PrettyPrintingOptions object +int processXMLPrettyPrinting(char** xml, int* length, PrettyPrintingOptions* ppOptions); /* process the pretty-printing on a valid xml string (no check done !!!). The ppOptions ARE NOT FREE-ED after processing. The method returns 0 if the pretty-printing has been done. */ +PrettyPrintingOptions* createDefaultPrettyPrintingOptions(void); /* creates a default PrettyPrintingOptions object */ #endif @@ Diff output truncated at 100000 characters. @@ -------------- This E-Mail was brought to you by github_commit_mail.py (Source: TBD).

1 0

[geany/geany-plugins] 2cb414: ShiftColumn: Remove superfluous semicolons
by Colomban Wendling 08 Jul '12

08 Jul '12

Branch: refs/heads/master Author: Colomban Wendling <ban(a)herbesfolles.org> Committer: Colomban Wendling <ban(a)herbesfolles.org> Date: Sun, 08 Jul 2012 20:43:21 Commit: 2cb41447340a5c7460efdda4972762f339be5fd6 https://github.com/geany/geany-plugins/commit/2cb41447340a5c7460efdda497276… Log Message: ----------- ShiftColumn: Remove superfluous semicolons Modified Paths: -------------- shiftcolumn/src/shiftcolumn.c Modified: shiftcolumn/src/shiftcolumn.c 4 files changed, 2 insertions(+), 2 deletions(-) =================================================================== @@ -45,10 +45,10 @@ GeanyData *geany_data; GeanyFunctions *geany_functions; -PLUGIN_VERSION_CHECK(130); +PLUGIN_VERSION_CHECK(130) PLUGIN_SET_INFO(_("Shift Column"), _("Shift a selection left and right"), - VERSION, "Andrew L Janke <a.janke(a)gmail.com>"); + VERSION, "Andrew L Janke <a.janke(a)gmail.com>") static GtkWidget *menu_item_shift_left = NULL; @@ Diff output truncated at 100000 characters. @@ -------------- This E-Mail was brought to you by github_commit_mail.py (Source: TBD).

1 0

[geany/geany-plugins] 2cb414: ShiftColumn: Remove superfluous semicolons
by Colomban Wendling 08 Jul '12

08 Jul '12

Branch: refs/heads/1.22_release Author: Colomban Wendling <ban(a)herbesfolles.org> Committer: Colomban Wendling <ban(a)herbesfolles.org> Date: Sun, 08 Jul 2012 20:43:21 Commit: 2cb41447340a5c7460efdda4972762f339be5fd6 https://github.com/geany/geany-plugins/commit/2cb41447340a5c7460efdda497276… Log Message: ----------- ShiftColumn: Remove superfluous semicolons Modified Paths: -------------- shiftcolumn/src/shiftcolumn.c Modified: shiftcolumn/src/shiftcolumn.c 4 files changed, 2 insertions(+), 2 deletions(-) =================================================================== @@ -45,10 +45,10 @@ GeanyData *geany_data; GeanyFunctions *geany_functions; -PLUGIN_VERSION_CHECK(130); +PLUGIN_VERSION_CHECK(130) PLUGIN_SET_INFO(_("Shift Column"), _("Shift a selection left and right"), - VERSION, "Andrew L Janke <a.janke(a)gmail.com>"); + VERSION, "Andrew L Janke <a.janke(a)gmail.com>") static GtkWidget *menu_item_shift_left = NULL; @@ Diff output truncated at 100000 characters. @@ -------------- This E-Mail was brought to you by github_commit_mail.py (Source: TBD).

1 0

[geany/geany-plugins] 3c3aa7: GeanyLipsum: Remove superfluous semicolon
by Colomban Wendling 08 Jul '12

08 Jul '12

Branch: refs/heads/master Author: Colomban Wendling <ban(a)herbesfolles.org> Committer: Colomban Wendling <ban(a)herbesfolles.org> Date: Sun, 08 Jul 2012 20:42:58 Commit: 3c3aa783e7c49a6f80f04e212525b46124bfe8d4 https://github.com/geany/geany-plugins/commit/3c3aa783e7c49a6f80f04e212525b… Log Message: ----------- GeanyLipsum: Remove superfluous semicolon Modified Paths: -------------- geanylipsum/src/geanylipsum.c Modified: geanylipsum/src/geanylipsum.c 2 files changed, 1 insertions(+), 1 deletions(-) =================================================================== @@ -40,7 +40,7 @@ _("GeanyLipsum"), _("Creating dummy text with Geany"), "0.4.3", - "Frank Lanitz <frank(a)frank.uvena.de>"); + "Frank Lanitz <frank(a)frank.uvena.de>") static GtkWidget *main_menu_item = NULL; static gchar *lipsum = NULL; @@ Diff output truncated at 100000 characters. @@ -------------- This E-Mail was brought to you by github_commit_mail.py (Source: TBD).

1 0

[geany/geany-plugins] 3c3aa7: GeanyLipsum: Remove superfluous semicolon
by Colomban Wendling 08 Jul '12

08 Jul '12

1 0

[geany/geany-plugins] 0ade23: GeanyGenDoc: Fix a few harmless warnings for GCC to shut up
by Colomban Wendling 08 Jul '12

08 Jul '12

Branch: refs/heads/master Author: Colomban Wendling <ban(a)herbesfolles.org> Committer: Colomban Wendling <ban(a)herbesfolles.org> Date: Sun, 08 Jul 2012 20:42:02 Commit: 0ade235521f02a434c412e0b3df4a5b0d1fbc57c https://github.com/geany/geany-plugins/commit/0ade235521f02a434c412e0b3df4a… Log Message: ----------- GeanyGenDoc: Fix a few harmless warnings for GCC to shut up Modified Paths: -------------- geanygendoc/src/ggd.c Modified: geanygendoc/src/ggd.c 10 files changed, 5 insertions(+), 5 deletions(-) =================================================================== @@ -340,7 +340,7 @@ comment = get_comment (ft, setting, doc, tag, &cursor_offset); if (comment) { - gint pos; + gint pos = 0; switch (setting->position) { case GGD_POS_AFTER: @@ -518,8 +518,8 @@ gboolean success = FALSE; const TMTag *tag = NULL; GPtrArray *tag_array = NULL; - GgdFileType *filetype; - GgdDocType *doctype; + GgdFileType *filetype = NULL; + GgdDocType *doctype = NULL; g_return_val_if_fail (DOC_VALID (doc), FALSE); @@ -573,8 +573,8 @@ const gchar *doc_type) { gboolean success = FALSE; - GgdFileType *filetype; - GgdDocType *doctype; + GgdFileType *filetype = NULL; + GgdDocType *doctype = NULL; g_return_val_if_fail (DOC_VALID (doc), FALSE); @@ Diff output truncated at 100000 characters. @@ -------------- This E-Mail was brought to you by github_commit_mail.py (Source: TBD).

1 0

[geany/geany-plugins] 0ade23: GeanyGenDoc: Fix a few harmless warnings for GCC to shut up
by Colomban Wendling 08 Jul '12

08 Jul '12

Branch: refs/heads/1.22_release Author: Colomban Wendling <ban(a)herbesfolles.org> Committer: Colomban Wendling <ban(a)herbesfolles.org> Date: Sun, 08 Jul 2012 20:42:02 Commit: 0ade235521f02a434c412e0b3df4a5b0d1fbc57c https://github.com/geany/geany-plugins/commit/0ade235521f02a434c412e0b3df4a… Log Message: ----------- GeanyGenDoc: Fix a few harmless warnings for GCC to shut up Modified Paths: -------------- geanygendoc/src/ggd.c Modified: geanygendoc/src/ggd.c 10 files changed, 5 insertions(+), 5 deletions(-) =================================================================== @@ -340,7 +340,7 @@ comment = get_comment (ft, setting, doc, tag, &cursor_offset); if (comment) { - gint pos; + gint pos = 0; switch (setting->position) { case GGD_POS_AFTER: @@ -518,8 +518,8 @@ gboolean success = FALSE; const TMTag *tag = NULL; GPtrArray *tag_array = NULL; - GgdFileType *filetype; - GgdDocType *doctype; + GgdFileType *filetype = NULL; + GgdDocType *doctype = NULL; g_return_val_if_fail (DOC_VALID (doc), FALSE); @@ -573,8 +573,8 @@ const gchar *doc_type) { gboolean success = FALSE; - GgdFileType *filetype; - GgdDocType *doctype; + GgdFileType *filetype = NULL; + GgdDocType *doctype = NULL; g_return_val_if_fail (DOC_VALID (doc), FALSE); @@ Diff output truncated at 100000 characters. @@ -------------- This E-Mail was brought to you by github_commit_mail.py (Source: TBD).

1 0

[geany/geany-plugins] 17105b: GeanySendMail: Remove useless \ before %
by Colomban Wendling 08 Jul '12

08 Jul '12

Branch: refs/heads/master Author: Colomban Wendling <ban(a)herbesfolles.org> Committer: Colomban Wendling <ban(a)herbesfolles.org> Date: Sun, 08 Jul 2012 18:05:48 Commit: 17105b17532f159f0f5cf8636e9d09de0f7d714b https://github.com/geany/geany-plugins/commit/17105b17532f159f0f5cf8636e9d0… Log Message: ----------- GeanySendMail: Remove useless \ before % Modified Paths: -------------- geanysendmail/src/geanysendmail.c Modified: geanysendmail/src/geanysendmail.c 10 files changed, 5 insertions(+), 5 deletions(-) =================================================================== @@ -313,12 +313,12 @@ GtkWidget *plugin_configure(GtkDialog *dialog) if (mailer != NULL) gtk_entry_set_text(GTK_ENTRY(pref_widgets.entry), mailer); - label2 = gtk_label_new(_("Note: \n\t\%f will be replaced by your file."\ - "\n\t\%r will be replaced by recipient's email address."\ - "\n\t\%b will be replaced by basename of a file"\ + label2 = gtk_label_new(_("Note: \n\t%f will be replaced by your file."\ + "\n\t%r will be replaced by recipient's email address."\ + "\n\t%b will be replaced by basename of a file"\ "\n\tExamples:"\ - "\n\tsylpheed --attach \"\%f\" --compose \"\%r\""\ - "\n\tmutt -s \"Sending \'\%b\'\" -a \"\%f\" \"\%r\"")); + "\n\tsylpheed --attach \"%f\" --compose \"%r\""\ + "\n\tmutt -s \"Sending \'%b\'\" -a \"%f\" \"%r\"")); gtk_label_set_selectable(GTK_LABEL(label2), TRUE); gtk_widget_show(label2); gtk_misc_set_alignment(GTK_MISC(label2), 0, 0.5); @@ Diff output truncated at 100000 characters. @@ -------------- This E-Mail was brought to you by github_commit_mail.py (Source: TBD).

1 0

[geany/geany-plugins] 17105b: GeanySendMail: Remove useless \ before %
by Colomban Wendling 08 Jul '12

08 Jul '12

Branch: refs/heads/1.22_release Author: Colomban Wendling <ban(a)herbesfolles.org> Committer: Colomban Wendling <ban(a)herbesfolles.org> Date: Sun, 08 Jul 2012 18:05:48 Commit: 17105b17532f159f0f5cf8636e9d09de0f7d714b https://github.com/geany/geany-plugins/commit/17105b17532f159f0f5cf8636e9d0… Log Message: ----------- GeanySendMail: Remove useless \ before % Modified Paths: -------------- geanysendmail/src/geanysendmail.c Modified: geanysendmail/src/geanysendmail.c 10 files changed, 5 insertions(+), 5 deletions(-) =================================================================== @@ -313,12 +313,12 @@ GtkWidget *plugin_configure(GtkDialog *dialog) if (mailer != NULL) gtk_entry_set_text(GTK_ENTRY(pref_widgets.entry), mailer); - label2 = gtk_label_new(_("Note: \n\t\%f will be replaced by your file."\ - "\n\t\%r will be replaced by recipient's email address."\ - "\n\t\%b will be replaced by basename of a file"\ + label2 = gtk_label_new(_("Note: \n\t%f will be replaced by your file."\ + "\n\t%r will be replaced by recipient's email address."\ + "\n\t%b will be replaced by basename of a file"\ "\n\tExamples:"\ - "\n\tsylpheed --attach \"\%f\" --compose \"\%r\""\ - "\n\tmutt -s \"Sending \'\%b\'\" -a \"\%f\" \"\%r\"")); + "\n\tsylpheed --attach \"%f\" --compose \"%r\""\ + "\n\tmutt -s \"Sending \'%b\'\" -a \"%f\" \"%r\"")); gtk_label_set_selectable(GTK_LABEL(label2), TRUE); gtk_widget_show(label2); gtk_misc_set_alignment(GTK_MISC(label2), 0, 0.5); @@ Diff output truncated at 100000 characters. @@ -------------- This E-Mail was brought to you by github_commit_mail.py (Source: TBD).

1 0

[geany/geany-plugins] f61a90: GeanySendMail: Use N_() for translations in arrays
by Colomban Wendling 08 Jul '12

08 Jul '12

Branch: refs/heads/master Author: Colomban Wendling <ban(a)herbesfolles.org> Committer: Colomban Wendling <ban(a)herbesfolles.org> Date: Sun, 08 Jul 2012 18:02:29 Commit: f61a90c92f4bcf1f5e5281a665ea9beaa3f77818 https://github.com/geany/geany-plugins/commit/f61a90c92f4bcf1f5e5281a665ea9… Log Message: ----------- GeanySendMail: Use N_() for translations in arrays _() is a function and it then can't be used to initialize an array element in C89, so use N_() that only marks the string for extracting without actually translating it. This is not an issue here since the translation will be done automatically later by GTK (using the GETTEXT_PATKAGE registered in the same array). Modified Paths: -------------- geanysendmail/src/geanysendmail.c Modified: geanysendmail/src/geanysendmail.c 2 files changed, 1 insertions(+), 1 deletions(-) =================================================================== @@ -184,7 +184,7 @@ static void add_stock_item(void) GtkIconSet *icon_set; GtkIconFactory *factory = gtk_icon_factory_new(); GtkIconTheme *theme = gtk_icon_theme_get_default(); - GtkStockItem item = { GEANYSENDMAIL_STOCK_MAIL, _("Mail"), 0, 0, GETTEXT_PACKAGE }; + GtkStockItem item = { GEANYSENDMAIL_STOCK_MAIL, N_("Mail"), 0, 0, GETTEXT_PACKAGE }; if (gtk_icon_theme_has_icon(theme, "mail-message-new")) { @@ Diff output truncated at 100000 characters. @@ -------------- This E-Mail was brought to you by github_commit_mail.py (Source: TBD).

1 0

← Newer
1
2
3
4
5
6
7
...
22
Older →

Jump to page:

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

2008

Plugins-Commits July 2012