Gehad's patches
Miika Turkia
miika.turkia at gmail.com
Fri Jun 6 08:35:37 PDT 2014
Here is this patch set with my sign-off and some whitespace clean-up
miika
On Thu, Jun 5, 2014 at 8:40 PM, Gehad Elrobey <gehadelrobey at gmail.com>
wrote:
> I tested the export on the daily build on windows, and played with it a
> little it works well till now though the theme directory is not copied to
> the install directory yet.
> Also the export dialog GUI needs some enhancements on windows.
>
> Dirk may I remind you to add the HTML themes to the installation directory
> on windows.
>
>
> On Thu, Jun 5, 2014 at 5:46 AM, Miika Turkia <miika.turkia at gmail.com>
> wrote:
>
>> On Wed, Jun 4, 2014 at 11:47 PM, Gehad <gehadelrobey at gmail.com> wrote:
>>
>>> The attached patch move the quote function to membuffer.c with some
>>> generalization as suggested.
>>> some wrapper functions fix up the parameters for both save-xml and
>>> save-html.
>>>
>>> I didn't add save-git to make the function simpler, the quote function
>>> in save-git is a little bit different, so I think it will complicate things
>>> up.
>>>
>>
>> White space issues here, I suggest you also install clang-format and run
>> the whitespace.pl before sending patches (clang-format is not on default
>> PATH on my Ubuntu, thus the addition to environment when running the
>> script, shown below):
>>
>> ---8<---
>> $ PATH=$PATH:/usr/lib/llvm-3.5/bin/ scripts/whitespace.pl `git diff
>> --name-only origin`
>> --- membuffer.c.old 2014-06-05 06:35:27.067346290 +0300
>> +++ membuffer.c 2014-06-05 06:38:44.638087151 +0300
>> @@ -96,7 +96,7 @@
>> return;
>> }
>>
>> - room = len+1;
>> + room = len + 1;
>> }
>> }
>>
>> @@ -174,7 +174,7 @@
>> udeg = -udeg;
>> sign = "-";
>> }
>> - put_format(b,"%s%s%u.%06u%s", pre, sign, FRACTION(udeg, 1000000),
>> post);
>> + put_format(b, "%s%s%u.%06u%s", pre, sign, FRACTION(udeg, 1000000),
>> post);
>> }
>>
>> void put_quoted(struct membuffer *b, const char *text, int is_attribute,
>> int is_html)
>> ---8<---
>>
>> also another patch for fixing the path in different platforms.
>>
>>
>> Good, now we need someone to test the HTML export at least in Windows.
>>
>> miika
>>
>>>
>>>
>>> On 06/04/2014 07:15 AM, Dirk Hohndel wrote:
>>>
>>>> On Tue, Jun 03, 2014 at 09:12:36PM +0300, Miika Turkia wrote:
>>>>
>>>>> From 684bdf422b466763045a25179b0ee022c9cb5cf3 Mon Sep 17 00:00:00
>>>>> 2001
>>>>> From: Gehad elrobey <gehadelrobey at gmail.com>
>>>>> Date: Mon, 2 Jun 2014 20:10:54 +0300
>>>>> Subject: [PATCH 6/7] HTML: Better quoting to the export strings
>>>>>
>>>>> Quote the location, buddy, suit, tags and notes.
>>>>> This prevents JS code from crashing.
>>>>>
>>>>> Signed-off-by: Gehad elrobey <gehadelrobey at gmail.com>
>>>>> Signed-off-by: Miika Turkia <miika.turkia at gmail.com>
>>>>> -char *replace_char(char *str, char replace, char *replace_by)
>>>>> +void put_quoted(struct membuffer *b, const char *text)
>>>>> {
>>>>> + const char *p = text;
>>>>> +
>>>>> + for (;;) {
>>>>> + const char *escape;
>>>>> +
>>>>> + switch (*p++) {
>>>>> + default:
>>>>> + continue;
>>>>> + case 0:
>>>>> + escape = NULL;
>>>>> + break;
>>>>> + case 1 ... 8:
>>>>> + case 11:
>>>>> + case 12:
>>>>> + case 14 ... 31:
>>>>> + escape = "?";
>>>>> + break;
>>>>> + case '<':
>>>>> + escape = "<";
>>>>> + break;
>>>>> + case '>':
>>>>> + escape = ">";
>>>>> + break;
>>>>> + case '&':
>>>>> + escape = "&";
>>>>> + break;
>>>>> + case '\'':
>>>>> + escape = "'";
>>>>> + break;
>>>>> + case '\"':
>>>>> + escape = """;
>>>>> + break;
>>>>> + case '\n':
>>>>> + escape = "<br>";
>>>>> + break;
>>>>> + }
>>>>> + put_bytes(b, text, (p - text - 1));
>>>>> + if (!escape)
>>>>> + break;
>>>>> + put_string(b, escape);
>>>>> + text = p;
>>>>> }
>>>>>
>>>> This is more or less verbatim the quote() function from save-xml.c.
>>>> Why not just call that function (or have a wrapper around it)?
>>>> I hate the idea of having basically identical quote twice.
>>>>
>>>> I took the other patches, this one I would like to see redone.
>>>>
>>>> /D
>>>>
>>>
>>> --
>>> Regards,
>>> Gehad Elrobey
>>>
>>>
>>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.hohndel.org/pipermail/subsurface/attachments/20140606/978a9301/attachment-0001.html>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0002-HTML-export-to-system-dependant-path-separator.patch
Type: text/x-patch
Size: 1007 bytes
Desc: not available
URL: <http://lists.hohndel.org/pipermail/subsurface/attachments/20140606/978a9301/attachment-0002.bin>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-HTML-Better-quoting-to-the-export-strings.patch
Type: text/x-patch
Size: 8515 bytes
Desc: not available
URL: <http://lists.hohndel.org/pipermail/subsurface/attachments/20140606/978a9301/attachment-0003.bin>
More information about the subsurface
mailing list