Commit Graph

3930 Commits

Author SHA1 Message Date
kortschak 3f8082d52e Merged in gardening/simplify (pull request #53)
rtmp: simplify rtmp code and make more type-safe

Approved-by: Saxon Milton <saxon.milton@gmail.com>
2018-09-09 23:32:44 +00:00
Dan Kortschak f0462ee511 rtmp: remove unused fields
reported by honnef.co/go/tools/cmd/unused

rtmp_headers.go:145:2: field sb_ssl is unused (U1000)
rtmp_headers.go:167:2: field subscribepath is unused (U1000)
rtmp_headers.go:168:2: field usherToken is unused (U1000)
rtmp_headers.go:170:2: field pubUser is unused (U1000)
rtmp_headers.go:171:2: field pubPasswd is unused (U1000)
rtmp_headers.go:173:2: field edepth is unused (U1000)
rtmp_headers.go:175:2: field stopTime is unused (U1000)
rtmp_headers.go:180:2: field pFlags is unused (U1000)
rtmp_headers.go:189:2: field bufpos is unused (U1000)
rtmp_headers.go:190:2: field buflen is unused (U1000)
rtmp_headers.go:191:2: field timestamp is unused (U1000)
rtmp_headers.go:195:2: field initialFrameType is unused (U1000)
rtmp_headers.go:197:2: field metaHeader is unused (U1000)
rtmp_headers.go:198:2: field initialFrame is unused (U1000)
rtmp_headers.go:199:2: field nMetaHeaderSize is unused (U1000)
rtmp_headers.go:200:2: field nInitialFrameSize is unused (U1000)
rtmp_headers.go:223:2: field m_mediaStamp is unused (U1000)
rtmp_headers.go:224:2: field m_pauseStamp is unused (U1000)
rtmp_headers.go:245:2: field m_polling is unused (U1000)
rtmp_headers.go:248:2: field m_clientID is unused (U1000)
2018-09-09 19:36:55 +09:30
Dan Kortschak fed685b107 rtmp: fix test 2018-09-08 16:07:00 +09:30
Dan Kortschak 834c2bc632 rtmp: remove C_AVal 2018-09-07 20:40:46 +09:30
Dan Kortschak dde20e1a7b rtmp: more simplification of url parsing 2018-09-07 19:41:55 +09:30
Dan Kortschak 9212bed0ed rtmp: simplify url parsing 2018-09-07 07:02:12 +09:30
Dan Kortschak bf2a2ec7a8 rtmp: remove C_AVal from C_RTMP_METHOD 2018-09-06 19:18:17 +09:30
Dan Kortschak 28f5ec6a47 rtmp: make isready return a bool 2018-09-06 18:40:47 +09:30
Dan Kortschak b1c904ee22 rtmp: reduce CAV/AVC conversions 2018-09-06 18:31:14 +09:30
Dan Kortschak 42f7316f09 rtmp: ensure we catch invalid methods 2018-09-06 18:06:45 +09:30
Dan Kortschak 569cc100fa retmp: simplify big switch 2018-09-06 18:05:01 +09:30
Dan Kortschak 97127030b3 rtmp: remove C_AVal from RTMP type 2018-09-06 16:52:48 +09:30
Dan Kortschak fed9d366c6 rtmp: remove C_Aval from C_AMFObjectProperty 2018-09-06 16:49:25 +09:30
Dan Kortschak c4800034a6 rtmp: remove C_AVal from P_vu 2018-09-06 16:10:15 +09:30
Dan Kortschak ed40f36ffe rtmp: remove C_AVal from C_AMF3ReadString
Note that this function is not used.
2018-09-06 15:57:09 +09:30
Dan Kortschak f005c31b44 rtmp: encode/decode typed boolean 2018-09-06 15:36:27 +09:30
Dan Kortschak 470e61a465 rtmp: remove C_AVal from encoder/decoder functions 2018-09-06 15:32:29 +09:30
Dan Kortschak 019a1f1c90 rtmp: make string decoding a returning function 2018-09-06 12:46:07 +09:30
Dan Kortschak a626a740ea rtmp: remove missed comments 2018-09-06 12:29:03 +09:30
Dan Kortschak aac0801cdb rtmp: use bytes.Equal for signature matching 2018-09-06 12:26:35 +09:30
Dan Kortschak 6c6ed78e8f rtmp: convert boolish ints to bool 2018-09-06 12:18:45 +09:30
Dan Kortschak f3a0968fb4 rtmp: port C_CloseInternal 2018-09-06 11:46:48 +09:30
Dan Kortschak 8674b23b28 rtmp: port C_SendFCUnpublish 2018-09-06 11:26:52 +09:30
Dan Kortschak b7c148fd93 rtmp: simplify body pointer setup 2018-09-06 11:11:29 +09:30
Dan Kortschak c31021149d rtmp: remove redundant comments 2018-09-06 11:04:08 +09:30
Saxon Milton 257544ef3f Merged in rtmp-unsafe-cleanup (pull request #52)
Rtmp unsafe cleanup

Approved-by: kortschak <dan@kortschak.io>
2018-09-05 03:40:53 +00:00
Dan Kortschak 7bfea781b4 rtmp: remove redundant size adjustment 2018-09-04 14:07:49 +09:30
saxon 078ba12093 rtmp: tried to replicate C behaviour by returning would be no of bytes written 2018-09-02 14:20:25 +09:30
saxon 8cf8f07b9c rtmp: corrected C_RTMP_Write c file line number 2018-09-02 13:40:18 +09:30
saxon 3d3ec6b893 rtmp: check that len(buf) is not less than 4 before buf = buf[4:] 2018-09-02 13:31:52 +09:30
saxon 6da0967b80 rtmp: removed check to see if byte slice length is less that 0 - which doesn't happen 2018-09-02 12:27:25 +09:30
saxon 9b152f0495 rtmp: removed math.Abs in C_RTMP_Write for checking noOfBytesRead 2018-09-02 11:35:32 +09:30
saxon 24968c83b3 rtmp: removed use of s2 and replaced with len(buf) 2018-09-02 11:32:11 +09:30
saxon 390c882792 rtmp: removed comments containing previous code in C_RTMP_Write 2018-09-02 10:21:10 +09:30
saxon 269c8176a1 rtmp: changed data slice to buf to simplify 2018-09-02 10:19:29 +09:30
saxon e861420e69 Merge branch 'master' into rtmp-unsafe-cleanup 2018-09-02 00:04:32 +09:30
saxon b3c25d3bc7 rtmp: initial commit 2018-09-02 00:03:30 +09:30
kortschak f2def3fd34 Merged in rtmp/unsafe-cleanup (pull request #51)
rtmp: unsafe cleanup

Approved-by: Saxon Milton <saxon.milton@gmail.com>
2018-09-01 14:22:17 +00:00
Dan Kortschak 4fa4dce908 rtmp: remove redundant reconversions 2018-09-01 23:35:28 +09:30
Dan Kortschak 05bed207b3 rtmp: remove unnecessary int conversions 2018-09-01 23:31:20 +09:30
Dan Kortschak 74e468dfb3 rtmp: remove unnecessary type conversions with unconvert 2018-09-01 23:31:08 +09:30
saxon 287c3d32ac rtmp: tried removing some unsafe pointers in C_RTMP_Write - ended up breaking it. Youtube tells me it's starting but it now goes live 2018-09-01 23:23:07 +09:30
Dan Kortschak 40785e7454 rtmp: make tests pass 2018-09-01 23:20:32 +09:30
Dan Kortschak c494da9a1d rtmp: remove indxBytePtr
Manual clean up.
2018-08-31 12:01:34 +09:30
Dan Kortschak e235da36a1 rtmp: simplify slice conversion
Second pass:

gofmt -w -r 'ptrToSlice(p,i)->(*[_Gi]byte)(p)[:i]' {rtmp,parseurl,amf}.go
2018-08-31 11:43:49 +09:30
Dan Kortschak 52c6b1f60a rtmp: simplify unsafe slice conversion
Mechanical:

gofmt -w -r 'ptrToSlice(unsafe.Pointer(p),i)->(*[_Gi]byte)(unsafe.Pointer(p))[:i]' {rtmp,parseurl,amf}.go
2018-08-31 11:42:22 +09:30
Dan Kortschak 0053034e94 rtmp: simplify indexing
This is a mechanical change in line with the previous:

gofmt -w -r '*indxBytePtr(p,i)->(*[_Gi]byte)(p)[i]' {rtmp,parseurl,amf}.go
2018-08-31 11:29:41 +09:30
Dan Kortschak 18e1e7fc45 rtmp: simplify indexing
This is a mechanical change with the addition of a constant:

gofmt -w -r '*indxBytePtr(unsafe.Pointer(p), i)->(*[_Gi]byte)(unsafe.Pointer(p))[i]' {rtmp,parseurl,amf}.go
2018-08-31 11:24:42 +09:30
Dan Kortschak 6187df1c20 rtmp: remove deadcode and make tests pass 2018-08-31 11:13:59 +09:30
Saxon Milton ddefd198d0 Merged in rtmp-structure-improvements (pull request #48)
rtmp: structure improvements

Approved-by: kortschak <dan@kortschak.io>
2018-08-30 11:06:44 +00:00