| Commit message (Collapse) | Author | Age | Files | Lines |
... | |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Client: go
Staticcheck is the recommended replacement of the frozen and deprecated
official golint linter [1].
Fix the things it complained about (or add lint:ignore directive) in:
- lib/go/thrift
- lib/go/test/tests
- tutorial/go/src
- test/go/src
- compiler generated code
The majority of the fixes are in the following categories:
- Use of deprecated function (mainly the TConfiguration related ones)
- Redundant break in switch cases
- Unused and unexported variables/fields/functions
Also in the same spirit as fb539ae, remove the error return from
NewTSSLSocket as it can never be non-nil.
This change will be cherry-picked into 0.15.0 branch after merged.
[1]: https://groups.google.com/g/golang-nuts/c/rCP70Aq_tBc
|
| |
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This change improves two problems in go code imports:
1. Always rename import the thrift package into "thrift", as we allow
the user to use a different library to replace the official one from
the compiler command line, this makes sure that in compiler generated
go code we can always blindly use "thrift.*".
2. We added auto rename import dedup in d9019fc5a4, but in that change
for system packages we always use the full import path as the dedup
identifier, so system package "database/sql/driver" would not be
detected as a conflict against a thrift go namespace of
"foo.bar.driver". Use the part after the last "/" in system packages
as the dedup identifier instead.
|
|
|
|
|
| |
Client: netstd
Patch: Jens Geyer
|
|
|
|
|
|
|
|
|
| |
with "_result" or "_args"
Client: netstd
Patch: Jens Geyer
This closes #2424
|
|
|
|
|
|
|
|
|
| |
Client: go
The current compiler will generate uncompilable code when we use
optional enum and/or typedef'd types in a thrift constant.
This fixes the issue, also adds a test for that.
|
|
|
|
|
|
|
| |
them public
Client: netstd
Patch: Jens Geyer
|
|
|
|
|
|
|
| |
Client: lua
Patch: Kaibin Huang
This closes #2413
|
| |
|
|
|
|
|
|
|
| |
message write phase
Client: Delphi
Patch: Jens Geyer
|
|
|
|
|
|
|
|
|
| |
TypeScript.
Client: node,js
Patch: Axel Jäger
This closes #2386
|
|
|
|
|
|
|
| |
Client: Lua
Patch: Jeffrey Han
This closes #2394
|
|
|
|
|
| |
Client: netstd
Patch: Jens Geyer
|
|
|
|
|
|
|
| |
Client: netstd
Patch: Jens Geyer
This closes #2381
|
|
|
|
|
|
|
| |
Client: netstd
Patch: Jens Geyer
This closes #2374
|
|
|
|
|
|
|
| |
Client: netstd
Patch: Jens Geyer
This closes #2372
|
|
|
|
|
|
|
| |
Client: hx
Patch: Jens Geyer
This closes #2370
|
|
|
|
|
|
|
| |
Client: cpp
Patch: Kashirin Alex
This closes #2318
|
|
|
|
|
|
|
| |
Client netstd
Patch: Thomas Bruggink
This closes #2364
|
|
|
|
|
|
|
| |
on Haxe cpp targets
Client: hx
Patch: Jens Geyer
|
|
|
|
|
|
|
| |
Client: erl
Patch: Sergey Yelin
This closes #2357
|
|
|
|
|
|
|
| |
Client: erl
Patch: Sergey Yelin
This closes #2354
|
|
|
|
|
|
|
| |
Client: hs
Patch: Jens Geyer
This closes #2352
|
|
|
|
|
|
|
| |
Client: haxe
Patch: Jens Geyer
This closes #2349
|
|
|
| |
Client: rs
|
|
|
| |
Client: rs
|
|
|
| |
Client: rs
|
|
|
|
|
| |
clippy-clean (#2341)
Client: rs
|
|
|
|
|
| |
values (#2337)
Client: rs
|
|
|
|
|
|
|
| |
Client: netstd
Patch: Jens Geyer
The recommended pattern matching expression \"that is not SomeType\" requires C#9/net5.0 but we still want to support earlier versions (.NET Core 3.1 is LTS until 2022-DEC-03)
|
|
|
|
|
|
|
|
|
|
| |
Client: go
When a thrift file includes 2 or more other thrift files, and those
included thrift files do not have explicit go namespaces defined, the
current import dedup logic would wrongly use their empty namespace and
skip the second one, while the real import namespace should be inferred
from the filename.
|
|
|
|
|
|
|
| |
Client: go
This fixes the bug reported in
https://github.com/apache/thrift/pull/2315#discussion_r577919697.
|
|
|
|
|
|
|
|
| |
compiler warning about always true comparisons
Patch: Orion Poplawski
This closes #2331
|
|
|
|
|
|
|
| |
Client: AS3
Patch: Jens Geyer
This closes #2329
|
|
|
|
|
| |
Client: hs
Patch: Jens Geyer
|
| |
|
|
|
|
|
| |
- This reverts commit 876aee506b937d64b1fb035ff65e5e52def04c04.
- This reverts commit b89316d54bb0253493b48446cb510755fb3196d2.
|
|
|
|
|
|
|
| |
- (NO TICKET) fix Haxe warning
- Referenzdaten PerfTest Delphi und netstd
- "custom build" added to version info
- netstd nuget upgrades
|
|
|
|
| |
Only compiler, test, lib and tutorial code.
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Client: go
There is a duplicate elements check for set in writeFields* function,
and it compares elements using reflect.DeepEqual which is expensive.
It's much faster that generates a *Equals* function for set elements and
call it in duplicate elements check, especially for nested struct
element.
Closes #2307.
|
| |
|
|
|
|
|
|
|
|
|
| |
Client: go
Make a breaking change so that TClient.Call returns the response
meta, currently only contains headers but could be expanded in the
future, and make a compiler change to compiler generated clients to take
advantage of that and provide access to response metadata to users.
|
|
|
|
|
|
|
|
|
|
|
|
| |
Client: go
Add TExceptionType enum type, and add
TExceptionType() TExceptionType
function to TException definition.
Also make TProtocolException unwrap-able.
|
|
|
|
|
| |
Client: netstd
Patch: Jens Geyer
|
|
|
|
|
|
|
| |
Client: netstd
Patch: Jens Geyer
This closes #2289
|
|
|
|
|
| |
Client: netstd
Patch: Jens Geyer
|
|
|
|
| |
Console.Error as before
|
|
|
|
|
|
|
| |
Client: netstd
Patch: Konstantin Pozdniakov & Jens Geyer
This closes #2284
|