Commit 2ae026c
committed
remove a circular require
If applied, this commit removes some circular `require`s.
Prior to this change, we had the following circular `require`s
```
require_relative 'entry' unless defined? Net::LDAP::Entry # dataset
require_relative 'dataset' unless defined? Net::LDAP::Dataset # entry
```
This works (both classes need each other in methods), but it's
unnecessary, since calling `require` twice does nothing, since
`$LOADED_FEATURES` has already been updated.
This change moves those `require`s to the toplevel, and removes the
`defined?` check.
---
No tests were added, since the existing tests passed without issue.1 parent 27eb85d commit 2ae026c
2 files changed
+4
-4
lines changed| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
| 1 | + | |
| 2 | + | |
1 | 3 | | |
2 | 4 | | |
3 | 5 | | |
| |||
164 | 166 | | |
165 | 167 | | |
166 | 168 | | |
167 | | - | |
168 | | - | |
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
| 1 | + | |
| 2 | + | |
1 | 3 | | |
2 | 4 | | |
3 | 5 | | |
| |||
195 | 197 | | |
196 | 198 | | |
197 | 199 | | |
198 | | - | |
199 | | - | |
0 commit comments