Commit Graph

57 Commits

Author SHA1 Message Date
Simon South
b62093db94 THRIFT-3285 c_glib: Library: Build with all warnings enabled
This matches the settings ("-Wall -Wextra -pedantic") used to build
the C++ library.
2015-08-03 19:07:42 +02:00
Simon South
31186c467a THRIFT-3285 c_glib: Library: Use AM_CPPFLAGS for include paths, not AM_CFLAGS
This follows the description of the two variables in the automake
documentation.
2015-08-03 19:07:42 +02:00
Jens Geyer
a2e0c5075d THRIFT-3260 multiple warnings in c_glib tutorial
Client: c_glib
Simon South <ssouth@simonsouth.com>

This closes #573

Here are additional changes that should really and truly resolve all the warnings generated when building the c_glib tutorial:

Compiler:
- Do not output a trailing comma in exception-enum definitions.
- Move variable declarations to avoid mixing declarations and code in generated code.
- Improve the readability of affected code blocks (and rely on indent_up and indent_down for indentation).
Library
- Use only C-style comments in headers included by clients.
Tutorial
- Move THRIFT_UNUSED_VAR calls to avoid mixing declarations and code.
2015-07-31 23:23:41 +02:00
Jens Geyer
482da722b9 THRIFT-3267 c_glib: "Critical" failure during unit tests
Client: c_glib
Patch: Simon South
2015-07-28 23:46:02 +02:00
Jens Geyer
1c1902788b THRIFT-3266 c_glib: Multiple compiler warnings building unit tests
Client: c_glib
Patch: Simon South

This closes #570
2015-07-28 23:15:18 +02:00
Jens Geyer
8c5fce9afa THRIFT-2917 "make clean" breaks test/c_glib
Client: c_glib
Patch: Simon South
2015-07-28 22:59:16 +02:00
Jens Geyer
8677719180 THRIFT-2957 warning: source file %s is in a subdirectory, but option 'subdir-objects' is disabled
Client: Compiler General
Patch: Simon South

This closes #563
2015-07-27 23:52:13 +02:00
Nobuaki Sukegawa
113b638790 THRIFT-3148 Markdown links to coding_standards are dead
This closes #491
2015-05-13 21:40:26 +02:00
Nobuaki Sukegawa
ef0a8fa62a THRIFT-3138 "make check" parallel execution is underministic 2015-05-10 18:25:56 +02:00
Jim King
79c9911b87 THRIFT-3084 add optional concurrent client limit enforcement to lib/cpp threaded servers 2015-04-30 19:48:15 +02:00
Roger Meier
3b99c970ad THRIFT-3106 CMake summary should give more information why a library is set to off
Patch: Pascal Bach
This closes #454
2015-04-20 22:49:48 +02:00
Marco Molteni
83494259d1 THRIFT-3099 cmake build is broken on FreeBSD 2015-04-16 22:23:27 +02:00
Roger Meier
8f27189dfd THRIFT-3090 cmake build is broken (set CMake version to 2.8.12) 2015-04-14 22:05:50 +02:00
Roger Meier
5af78c8776 Revert "THRIFT-3014 AppVeyor support - fix format"
This reverts commit 81b3c440c6.

commit included modified CMakeLists.txt files, as part from local test
2015-04-12 21:43:12 +02:00
Roger Meier
81b3c440c6 THRIFT-3014 AppVeyor support - fix format 2015-04-12 21:06:11 +02:00
Roger Meier
79e971e7de c_glib: fix some warnings 2015-04-12 13:11:33 +02:00
Konrad Grochowski
3bc6424c76 THRIFT-2724 - Coding standards template added all over project
Client: All
Patch: Konrad Grochowski

This closes #371
2015-02-05 12:15:19 +01:00
Jens Geyer
8bcfdd98ae THRIFT-2886 Integrate binary type in standard Thrift cross test
Client: C_glib, C++, D, Erlang, Go, Haskell, Lua, Java/Me, JavaScript, Node, Ocaml, Perl, PHP, Python, Ruby
Patch: Jens Geyer

This closes #341

Minimal server-side implementations and TODO stubs for various languages to let "make check" succeeed.
Not contained in this patch and still TODO:
- client side implementations, i.e. calls to testBinary() and appropriate tests
- server side hex printout missing for some languages
2015-01-03 17:37:54 +01:00
Pascal Bach
d5f87e1f21 CMake: Add CMake support for Thrift libraries
Currently the following libraries are supported:
- C++
- C_Glib
- Java (using Ant wrapper)

The compilers CMake file is adjusted to work with the new global CMakeLists.txt file.

Signed-off-by: Roger Meier <r.meier@siemens.com>
Signed-off-by: Pascal Bach <pascal.bach@siemens.com>
Also-by: Sergei Nikulov <sergey.nikulov@gmail.com>
2014-12-13 22:02:29 +01:00
Anatol Pomozov
4bf97c196a Do not drop CFLAGS from c_glib compilation
CFLAGS contains important flags. Dropping it introduces e.g. such issues:
https://sourceware.org/bugzilla/show_bug.cgi?id=13979

Signed-off-by: Anatol Pomozov <anatol.pomozov@gmail.com>

This closes #284
2014-12-02 00:10:11 +01:00
Roger Meier
3fa5d89e16 c_glib: remove -Werror 2014-11-02 12:19:54 +01:00
Roger Meier
035870d333 c_glib: fix automake dependencies 2014-10-22 23:43:05 +02:00
Roger Meier
81a1f996bc THRIFT-2602 fix missing dist files
- add automake 1.13 dependency to configure.ac and doc
- use serial-tests instead of .NOTPARALLEL (introduced by THRIFT-1829)

Patch: Roger Meier
2014-10-22 14:09:43 +02:00
Jens Geyer
79f988c27a THRIFT-2768: Whitespace Fixup
Client: General (Makefile.am)
Patch: Jens Geyer

Reverted makefile.am - tabs are not optional there
2014-10-03 20:42:54 +02:00
Jens Geyer
aaa8947f3a THRIFT-2768: Whitespace Fixup
Client: General (Makefile.am, *.thrift)
Patch: Jens Geyer

DocTest.thrift has NOT been changed, the trailing whitespaces are part of the test case.
2014-10-03 20:30:38 +02:00
Roger Meier
63243c6a29 THRIFT-2709 c_glib: Support server implementation
Patch: Simon South
2014-09-29 20:29:58 +02:00
Roger Meier
0cc6d3c7bd THRIFT-2690
c_glib: Incorrect behaviour when serializing a map with typedef'd keys

Patch: Simon South
2014-09-04 00:24:17 +02:00
Roger Meier
7fa9848b85 THRIFT-2680 c_glib: ThriftFramedTransport fails when peer unexpectedly closes connection
Patch: Simon South
2014-09-01 20:21:33 +02:00
Roger Meier
60b7ad6b3d THRIFT-2639 c_glib: Expose as properties members of generated structs
Patch: Simon South

Signed-off-by: Roger Meier <roger@apache.org>
2014-07-29 23:25:53 +02:00
Roger Meier
909f186fa6 THRIFT-2636 c_glib: ThriftApplicationException: Expose "type" and "message" properties
Patch: Simon South
2014-07-24 23:52:14 +02:00
Roger Meier
1d7e35a7cb THRIFT-2631 c_glib: Support service inheritance in clients
Patch: Simon South
2014-07-22 21:56:12 +02:00
Roger Meier
5c80b56573 THRIFT-2475 c_glib: buffered_transport_write function return always TRUE.
Patch: Jaesang Kim
2014-05-04 22:19:46 +02:00
Jens Geyer
6d1a83aa48 THRIFT-2506 Update TProtocolException error codes to be used consistently throughout the library
Patch: Jens Geyer
2014-05-03 00:49:05 +02:00
Jens Geyer
2f75e2daa3 THRIFT-2460 unopened socket fd must be less than zero.
Client: C glib
Patch: Jaesang Kim
2014-04-15 21:14:04 +02:00
Roger Meier
bea3144a45 THRIFT-2454: c_glib: There is no gethostbyname_r() in some OS
Patch: Jin-wook Jeong
2014-04-08 23:52:01 +02:00
Roger Meier
bf0b6291f8 THRIFT-2414 c_glib fix several bug
Patch: Jaesang Kim
2014-03-26 23:32:34 +01:00
Roger Meier
16fcad0b25 THRIFT-2407 use markdown (rename README => README.md)
Patch: Roger Meier
2014-03-19 06:47:47 +01:00
Roger Meier
62fb17d308 THRIFT-2138 c_glib: pkgconfig file created with wrong include path
Patch: Roger Meier
2013-08-26 21:50:16 +02:00
Roger Meier
a51186b7f2 THRIFT-1879 Add support for GObject-Introspection
Patch: Evan Nemerson
patch 1: 0001-Move-typedefs-before-public-structs-to-work-around-g.patch
2013-03-22 17:12:03 +01:00
Roger Meier
c095919087 THRIFT-1829 cpp unit tests fail to build using multiple make jobs
Patch: David Rennalls and Roger Meier
2013-01-15 23:20:19 +01:00
Roger Meier
e3da768367 THRIFT-1826 update c_glib source header paths
Patch: Simon South
2013-01-11 11:41:53 +01:00
Roger Meier
0193149842 THRIFT-1801 Sync up TApplicationException codes across languages and thrift implementations
Patch: Andrew Cox
2012-12-22 21:31:03 +01:00
Roger Meier
771ca3cdf1 THRIFT-1414 bufferoverflow in c_glib buffered transport/socket client
Patch: Christian Zimnick



git-svn-id: https://svn.apache.org/repos/asf/thrift/trunk@1399452 13f79535-47bb-0310-9956-ffa450edef68
2012-10-17 22:05:44 +00:00
Roger Meier
908d2f7839 THRIFT-1116 Compilation error in file TSSLSocket.cpp on FreeBSD-8.0
Patch: Pavlin Radoslavov


git-svn-id: https://svn.apache.org/repos/asf/thrift/trunk@1362896 13f79535-47bb-0310-9956-ffa450edef68
2012-07-18 12:03:20 +00:00
Roger Meier
c75797d906 THRIFT-1583 c_glib leaks memory
THRIFT-1578 C_GLib generated code does not compile
THRIFT-1582 Bad includes of nested thrift files in c_glib

Patch: José Antonio Santos
Minor Modificatons by roger:
- thrift prefix for includes
- g_object_unref instead of g_clear_object (needs glib>=2.28)

git-svn-id: https://svn.apache.org/repos/asf/thrift/trunk@1331752 13f79535-47bb-0310-9956-ffa450edef68
2012-04-28 11:33:58 +00:00
Roger Meier
49ff8b123a THRIFT-1552 Include paths for c/c++ should be prefixed with 'thrift/'
To ensure there are no include path collisions the C and C++ header
include paths should include 'thrift' as the root leaf. This will
prevent having to place /usr/include/thrift into the compilers include
header search path, which might otherwise result in the compiler
accidentally picking up headers that it shouldn't.

e.g. #include <foo/bar.h> should be #include <thrift/foo/bar.h>

Change-Id: I48f2b0f549bda0fc81e85506ac857adc800b98a1

git-svn-id: https://svn.apache.org/repos/asf/thrift/trunk@1325674 13f79535-47bb-0310-9956-ffa450edef68
2012-04-13 09:12:31 +00:00
Jake Farrell
363f6d414b THRIFT-1128:MAC OS X thrift.h incompatibility with Thrift.h
Client: cpp, c_glib
Patch: Jake Farrell

Moving c_glib headers into $(includedir)/thrift/c_glib to not collide with cpp headers.



git-svn-id: https://svn.apache.org/repos/asf/thrift/trunk@1296038 13f79535-47bb-0310-9956-ffa450edef68
2012-03-02 02:50:50 +00:00
Jake Farrell
e0c5316736 Thrift-1426: Dist package missing files for release 0.8
Client: build
Patch: Jake Farrell

Cleans up dist target and removes extra folders.



git-svn-id: https://svn.apache.org/repos/asf/thrift/trunk@1202686 13f79535-47bb-0310-9956-ffa450edef68
2011-11-16 12:58:36 +00:00
Roger Meier
345ecc739a THRIFT-690 add new ApplicationExceptions (INTERNAL_ERROR, PROTOCOL_ERROR) to the following languages: as3, c_glib, cocoa, cpp, erl, hs, js, nodejs, ocaml, perl, py, rb
git-svn-id: https://svn.apache.org/repos/asf/thrift/trunk@1153412 13f79535-47bb-0310-9956-ffa450edef68
2011-08-03 09:49:27 +00:00
Bryan Duxbury
3c9530c3fa THRIFT-1218. c_glib uses wrong name in pkg-config
Patch: Eric Evans

git-svn-id: https://svn.apache.org/repos/asf/thrift/trunk@1139079 13f79535-47bb-0310-9956-ffa450edef68
2011-06-23 21:20:34 +00:00