Hi,
On Mon, Nov 13, 2017 at 3:40 PM, David Sommerseth wrote:
> On 13/11/17 14:44, François Kooman wrote:
> > On 11/13/2017 01:16 PM, David Sommerseth wrote:
> [...snip...]
> >> But we should consider if we want to make use of a JSON library
> >> producing the
On 13/11/17 14:44, François Kooman wrote:
> On 11/13/2017 01:16 PM, David Sommerseth wrote:
[...snip...]
>> But we should consider if we want to make use of a JSON library
>> producing the JSON streams. The reason is to ensure the output is
>> according to the specification and that escaping if
Hi,
Thanks for the v2
On Mon, Nov 13, 2017 at 4:49 AM, Simon Rozman wrote:
> Data size arithmetic was reviewed according to 64-bit MSVC complaints.
>
> The warnings were addressed by migrating to size_t, rewriting the code,
> or silencing the warnings by an explicit cast where
On 11/13/2017 01:16 PM, David Sommerseth wrote:
> CSV just looks structured - until the layout is changed. While JSON
> (and even XML for that matter, _not_ saying we should have XML support!)
> is not strictly bound to the order of fields, so adding new fields is a
> no-brainer with JSON
Hi,
> > const char *p = _tcsrchr(find->cFileName, TEXT('.'));
> > return p && p != find->cFileName && !_tcsicmp(p+1, ext);
> >
> > No _tcslen() and no offending size variables.
> > And that convoluted original is then gone.
>
> The resulting code would look a lot nicer indeed. Stay tuned
On 12/11/17 09:34, Heiko Hund wrote:
> With the private gc_arena we do not have to allocate the strings
> found during parsing again, since we know the arena they are
> allocated in is valid as long as the argv vector is.
>
> Signed-off-by: Heiko Hund
> ---
>
On 12/11/17 09:27, Heiko Hund wrote:
> Prevent the re-allocations of memory when the internal argv grows
> beyond 2 and 4 arguments by initially allocating argv to hold up to
> 7 (+ trailing NULL) pointers.
>
> While at it rename argv_reset to argv_free to actually express
> what's going on. Redo
On 13/11/17 11:15, Steffan Karger wrote:
[...snip...]
>>> You know, the funny thing is that my main reason for implementing JSON
>>> support for status was to have it easily _machine_ readable, for higher
>>> level programming languages where JSON is "first class citizen", and not
>>> primarily
Hey,
Thanks a lot for putting a renewed effort into this. I've reviewed
this and compile tested with GCC 4.8.5, 6.3.1 and 7.2.1 - as there
are some warnings appearing; I'll come back to them in a bit.
On 12/11/17 09:24, Heiko Hund wrote:
> The previous implementation had the problem that it
Hi,
Since I don't use the mgmt interface / status file a lot, I didn't chip
in earlier. But since this quite closely matches my initial thoughts
I'll do so anyway.
On 13-11-17 10:54, Antonio Quartulli wrote:
> On 13/11/17 17:44, François Kooman wrote:
>> On 11/12/2017 10:42 PM, Gert Doering
On 12/11/17 01:14, Gert van Dijk wrote:
> I think this was omitted in 66bf378e.
>
> Signed-off-by: Gert van Dijk
Acked-by: Antonio Quartulli
We definitely need this otherwise the doxyfile won't leave you alone.
Thanks!
--
Antonio Quartulli
Hi,
On 13/11/17 17:44, François Kooman wrote:
> On 11/12/2017 10:42 PM, Gert Doering wrote:
>> On Sun, Nov 12, 2017 at 12:21:06PM -0500, Selva wrote:
>>> One of the niceties of JSON is its readability which is greatly reduced
>>> if formatted without line breaks.
>>
>> +1
>>
>> the difference in
Data size arithmetic was reviewed according to 64-bit MSVC complaints.
The warnings were addressed by migrating to size_t, rewriting the code,
or silencing the warnings by an explicit cast where appropriate.
---
src/openvpnserv/automatic.c | 20
On 11/12/2017 10:42 PM, Gert Doering wrote:
> On Sun, Nov 12, 2017 at 12:21:06PM -0500, Selva wrote:
>> One of the niceties of JSON is its readability which is greatly reduced
>> if formatted without line breaks.
>
> +1
>
> the difference in efficienty is not large, but "human readability for
>
Hi,
> Some of these changes are of dubious value as the string lengths involved
> are guaranteed to be small and there is no scope for overflow. And casting
> only stops the compiler warning, not potential overflow, if any..
Exactly. Where there's no scope for an overflow and compiler is too
15 matches
Mail list logo