Commit Graph

791 Commits

Author SHA1 Message Date
Paul Pacheco
a5da7d55d3 Remove empty lines 2018-10-13 12:27:00 -05:00
Paul Pacheco
0cc0fc0447 No reason for synclists to be sealed 2018-10-13 12:21:38 -05:00
Paul Pacheco
425289fd3d remove blank lines 2018-10-13 12:21:15 -05:00
Paul Pacheco
0fc75eb04d spacing issues 2018-10-13 12:17:13 -05:00
vis2k
3e751b3da5 Removed NetworkTransformPreview because no one uses it and it had random errors (#60) 2018-10-13 14:11:21 +02:00
vis2k
a2796c98c6 Removed NetworkTransformVisualizer component. Nobody uses it and what we really need is a decent NetworkTransform component with a simple OnDrawGizmos function instead. 2018-10-13 14:08:52 +02:00
Paul Pacheco
d5c407fa60 Synchronize synclists with onserialize/ondeserialize (callback) (#67)
* Synchronize synclists with onserialize/ondeserialize

* Add test to cover set

* Add tests for remove at and remove

* Simplify tests

* Test SyncListInt

* remove non standard, redundant count attribute

* Refactor tests

* Test the callback

* Test Count and isReadOnly

* Check that dirty bit is working

* removed unused method

* Remove empty line

* Don't track changes if this is not a server object

* We no longer have a registersynclistdelegate

* Fix issue with list not registering

* Explain what InitSyncObject is doing

* Check if field implements SyncObject instead of synclist

* Initialize lists in constructor instead of Awake

* Use Linq to simplify the code

* Remove extra space

* Remove extra line

* remove extra space

* Add brackets to ifs

* Remove exta spaces

* Clean up spacing

* Remove unnecesary method

* Remove unnecesry method

* Reduce cognitive complexity

* Remove space

* Remove extra spacing

* Typo, and make variable local

* Renamed parameter for clarity

* Fix indentation

* callback works on client and server and remove the dependency
2018-10-13 13:33:32 +02:00
Paul Pacheco
ed4060ac9a Remove extra blank line 2018-10-12 16:01:08 -05:00
Paul Pacheco
0772d44761 Nobody should instantiate LogFilter 2018-10-12 15:59:44 -05:00
Paul Pacheco
5879d24cb0 Remove extra semicolons 2018-10-12 15:59:01 -05:00
Paul Pacheco
6356e411be Spacing issues 2018-10-12 14:14:25 -05:00
Paul Pacheco
52ca8b746a Remove extra line 2018-10-12 13:17:53 -05:00
Paul Pacheco
8c9b937091 Remove extra semicolon 2018-10-12 13:16:43 -05:00
Paul Pacheco
1a326d0957 Do not initialize variables to default values, found by SA 2018-10-12 11:30:43 -05:00
Paul Pacheco
f15888dfef Removed unused parameters 2018-10-12 11:25:02 -05:00
Paul Pacheco
c258585cb2
Fix code style issues
Fix code styles issues found by SA
2018-10-12 11:08:16 -05:00
Paul Pacheco
3512903ec3
Fix invalid link
this is not a link
2018-10-11 21:06:34 -05:00
Paul Pacheco
9c874fa6e8
Fix bullet point indentation 2018-10-11 21:04:24 -05:00
Paul Pacheco
7317637682
Fix bullet indentation
Static analysis keeps bitching about this
2018-10-11 21:03:10 -05:00
Paul Pacheco
49b3f2bcaa
Fix bullet indentation
Static analysis keeps bitching about this.
2018-10-11 20:56:21 -05:00
Paul Pacheco
6684851467 Make variable local, keep less state 2018-10-11 20:24:34 -05:00
Paul Pacheco
7750ab664c Make variable local 2018-10-11 20:23:00 -05:00
Paul Pacheco
5a1be98885 initialize these lists only once 2018-10-11 20:19:57 -05:00
Paul Pacheco
a4b642fca6 These fields should never be assigned after creation 2018-10-11 20:16:35 -05:00
Paul Pacheco
875115db2c Remove redundant parenthesis 2018-10-11 20:13:54 -05:00
Paul Pacheco
b57bbf4ec7 ensure reader is only initialized once 2018-10-11 20:12:01 -05:00
Paul Pacheco
71ea8291d4 Make sure m_MessageHandlers are initialized only once 2018-10-11 20:11:06 -05:00
Paul Pacheco
8716eb6cdf remove redundant array size spec 2018-10-11 20:09:14 -05:00
Paul Pacheco
1b81fea8ef Channel is only set at constructor 2018-10-11 19:58:08 -05:00
Paul Pacheco
e2577ef785 Simplify conditionals 2018-10-11 19:43:20 -05:00
Paul Pacheco
9a542e98cc Simplify conditional 2018-10-11 19:40:36 -05:00
Paul Pacheco
e265ef3efd Remove useless assignment 2018-10-11 19:32:11 -05:00
Paul Pacheco
bb237d6fbb Simplify conditional 2018-10-11 19:31:39 -05:00
Paul Pacheco
5bd90fa63c Removed useless assignment 2018-10-11 19:30:42 -05:00
Paul Pacheco
680d6437af Simplify conditional 2018-10-11 19:29:25 -05:00
Paul Pacheco
31d39744e8 Remove duplicate lines 2018-10-10 13:53:40 -05:00
Paul Pacheco
f1efb6b889 Implement IEquatable as recommended by Microsoft and SA 2018-10-10 09:51:27 -05:00
Paul Pacheco
5b0d1190b8 Implement IEquatable as recommended by Microsoft and SA 2018-10-10 09:43:00 -05:00
Paul Pacheco
9cd74a240f Removed unused method 2018-10-09 23:59:15 -05:00
Paul Pacheco
f0723e50dd simplify with null coalesce 2018-10-09 23:53:02 -05:00
Paul Pacheco
059730ca37 Remove redundant casting 2018-10-09 14:48:04 -05:00
Paul Pacheco
757cafe0a7 Remove unused variable 2018-10-09 14:42:54 -05:00
Paul Pacheco
6a80f77e6d
fix list indentation 2018-10-09 14:20:36 -05:00
Paul Pacheco
216504a78c Clientscene is a static class (#59) 2018-10-09 19:56:04 +02:00
Paul Pacheco
4b2ab9b3d6 Remove redundant return 2018-10-09 11:11:07 -05:00
Paul Pacheco
6bbe38bbca Remove redundant return 2018-10-09 10:53:39 -05:00
Paul Pacheco
c1a68792c9 Ensure writer is initialized only once 2018-10-09 10:47:36 -05:00
Paul Pacheco
95b6e9fc12 Removed unused code 2018-10-09 10:39:00 -05:00
Paul Pacheco
95972329af Simplify validation 2018-10-09 10:33:12 -05:00
Paul Pacheco
c590b89836 Dont assign in conditional, it is confusing 2018-10-09 10:30:31 -05:00