Jump to content
Search In
  • More options...
Find results that contain...
Find results in...
printz

Eureka editor has bugs when splitting sectors

Recommended Posts

DISCLAIMER: I'm using the latest SVN build.

What I'm saying is that when I add a linedef to cross a sector in order to split it, Eureka sometimes (but constantly, for that sector) creates a solid (1-sided) wall, and I need to add the second sidedef (which thankfully references the correct new sector), or creates a 0-sided line, without creating any new sector, in which case I don't know what to do. Is this a bug that needs to be solved by the developer or is there a more suitable way to create linedefs to split sectors?

Share this post


Link to post

The developer would like know exactly how to reproduce the problem, ideally a wad file (if cannot do it on MAP01 of DOOM2) and the *exact* steps to reproduce.

Share this post


Link to post
andrewj said:

The developer would like know exactly how to reproduce the problem, ideally a wad file (if cannot do it on MAP01 of DOOM2) and the *exact* steps to reproduce.

yeah this sounds like a developer problem, might want to email somebody about it. this would be an annoying problem if i came across it.

Share this post


Link to post

Well I have a WIP (for Heretic). Unfortunately what I can say is try fooling around in the level until it happens. For example try drawing a linedef across a corridor by pressing space on a wall and space on the other wall. Often it will generate a solid line (which may even alter one of the sectors) instead of a 2s one.

https://www.dropbox.com/s/m1gonttsr877tmt/e1m1.wad?dl=0

Iwad: vanilla Heretic 1.3 sotsr.
E1m1.

Share this post


Link to post
andrewj said:

Ok, will experiment with that map and see if I can reproduce it...


I would like to help too, but im currently working, ill see if i can help when i get to my home pc, cant wait to see a solution / results from andrewj or printz himself.

Share this post


Link to post

This map has 3 vertex pairs that are not connected (i.e. sitting on top of each other).

Use the map checking functions to find them, it can even automatically merge them (though I suggest check each situation first).

Conclusion: not an editor problem.

Share this post


Link to post

I loaded the map into GZDB and let Map Analysis do it's thing.
While a few items may be trivial, like the wall torches and unused textures, the sector and vertex problems are not.

This is the result:

Spoiler

    Sector 1 is not closed
    Sector 21 is not closed
    Sector 27 is not closed
    Sector 51 has 0 sidedefs
    Sector 51 has no ceiling flat.
    Sector 51 has no floor flat.
    Sidedef 1 has unused upper texture "SQPEB1"
    Sidedef 113 has unused lower texture "SQPEB1"
    Sidedef 113 has unused upper texture "SQPEB1"
    Sidedef 116 has unused lower texture "SANDSQ2"
    Sidedef 116 has unused upper texture "SANDSQ2"
    Sidedef 119 has unused lower texture "SANDSQ2"
    Sidedef 119 has unused upper texture "SANDSQ2"
    Sidedef 13 has unused lower texture "CSTLRCK"
    Sidedef 13 has unused upper texture "CSTLRCK"
    Sidedef 131 has unused lower texture "SQPEB1"
    Sidedef 131 has unused upper texture "SQPEB1"
    Sidedef 151 has unused lower texture "CSTLRCK"
    Sidedef 151 has unused upper texture "CSTLRCK"
    Sidedef 155 has unused lower texture "CSTLRCK"
    Sidedef 155 has unused upper texture "CSTLRCK"
    Sidedef 158 has unused lower texture "CSTLRCK"
    Sidedef 158 has unused upper texture "CSTLRCK"
    Sidedef 16 has unused upper texture "SQPEB1"
    Sidedef 162 has unused lower texture "CSTLRCK"
    Sidedef 162 has unused upper texture "CSTLRCK"
    Sidedef 166 has unused lower texture "CSTLRCK"
    Sidedef 166 has unused upper texture "CSTLRCK"
    Sidedef 17 has unused lower texture "SQPEB1"
    Sidedef 17 has unused upper texture "SQPEB1"
    Sidedef 170 has unused lower texture "CSTLRCK"
    Sidedef 170 has unused upper texture "CSTLRCK"
    Sidedef 175 has unused lower texture "CSTLRCK"
    Sidedef 175 has unused upper texture "CSTLRCK"
    Sidedef 180 has unused lower texture "CSTLRCK"
    Sidedef 180 has unused upper texture "CSTLRCK"
    Sidedef 184 has unused lower texture "CSTLRCK"
    Sidedef 184 has unused upper texture "CSTLRCK"
    Sidedef 188 has unused lower texture "CSTLRCK"
    Sidedef 188 has unused upper texture "CSTLRCK"
    Sidedef 19 has unused lower texture "CSTLRCK"
    Sidedef 19 has unused upper texture "CSTLRCK"
    Sidedef 192 has unused lower texture "CSTLRCK"
    Sidedef 192 has unused upper texture "CSTLRCK"
    Sidedef 196 has unused lower texture "CSTLRCK"
    Sidedef 196 has unused upper texture "CSTLRCK"
    Sidedef 199 has unused upper texture "SQPEB1"
    Sidedef 2 has unused lower texture "SQPEB1"
    Sidedef 2 has unused upper texture "SQPEB1"
    Sidedef 20 has unused lower texture "CSTLRCK"
    Sidedef 20 has unused upper texture "CSTLRCK"
    Sidedef 200 has unused lower texture "SQPEB1"
    Sidedef 200 has unused upper texture "SQPEB1"
    Sidedef 201 has unused upper texture "SQPEB1"
    Sidedef 202 has unused lower texture "SQPEB1"
    Sidedef 202 has unused upper texture "SQPEB1"
    Sidedef 205 has unused lower texture "SANDSQ2"
    Sidedef 205 has unused upper texture "SANDSQ2"
    Sidedef 209 has unused lower texture "CSTLRCK"
    Sidedef 209 has unused upper texture "CSTLRCK"
    Sidedef 212 has unused lower texture "SANDSQ2"
    Sidedef 212 has unused upper texture "SANDSQ2"
    Sidedef 223 has unused lower texture "SANDSQ2"
    Sidedef 223 has unused upper texture "SANDSQ2"
    Sidedef 224 has unused lower texture "SANDSQ2"
    Sidedef 224 has unused upper texture "SANDSQ2"
    Sidedef 225 has unused lower texture "SANDSQ2"
    Sidedef 225 has unused upper texture "SANDSQ2"
    Sidedef 226 has unused lower texture "SANDSQ2"
    Sidedef 226 has unused upper texture "SANDSQ2"
    Sidedef 227 has unused upper texture "ORNGRAY"
    Sidedef 228 has unused lower texture "CTYSTUC4"
    Sidedef 228 has unused upper texture "CTYSTUC4"
    Sidedef 229 has unused upper texture "ORNGRAY"
    Sidedef 230 has unused lower texture "CTYSTUC4"
    Sidedef 230 has unused upper texture "CTYSTUC4"
    Sidedef 231 has unused upper texture "ORNGRAY"
    Sidedef 232 has unused lower texture "CTYSTUC4"
    Sidedef 232 has unused upper texture "CTYSTUC4"
    Sidedef 233 has unused upper texture "ORNGRAY"
    Sidedef 234 has unused lower texture "CTYSTUC4"
    Sidedef 234 has unused upper texture "CTYSTUC4"
    Sidedef 235 has unused upper texture "ORNGRAY"
    Sidedef 236 has unused lower texture "CTYSTUC4"
    Sidedef 236 has unused upper texture "CTYSTUC4"
    Sidedef 237 has unused upper texture "SANDSQ2"
    Sidedef 238 has unused lower texture "SANDSQ2"
    Sidedef 238 has unused upper texture "SANDSQ2"
    Sidedef 239 has unused upper texture "SANDSQ2"
    Sidedef 24 has unused upper texture "SQPEB1"
    Sidedef 240 has unused lower texture "SANDSQ2"
    Sidedef 240 has unused upper texture "SANDSQ2"
    Sidedef 241 has unused upper texture "SANDSQ2"
    Sidedef 242 has unused lower texture "SANDSQ2"
    Sidedef 242 has unused upper texture "SANDSQ2"
    Sidedef 25 has unused lower texture "SQPEB1"
    Sidedef 25 has unused upper texture "SQPEB1"
    Sidedef 272 has unused upper texture "GRSTNPBV"
    Sidedef 273 has unused lower texture "SANDSQ2"
    Sidedef 273 has unused upper texture "SANDSQ2"
    Sidedef 274 has unused upper texture "GRSTNPBV"
    Sidedef 275 has unused lower texture "SANDSQ2"
    Sidedef 275 has unused upper texture "SANDSQ2"
    Sidedef 276 has unused upper texture "GRSTNPBV"
    Sidedef 277 has unused lower texture "SANDSQ2"
    Sidedef 277 has unused upper texture "SANDSQ2"
    Sidedef 278 has unused upper texture "GRSTNPBV"
    Sidedef 279 has unused lower texture "SANDSQ2"
    Sidedef 279 has unused upper texture "SANDSQ2"
    Sidedef 280 has unused upper texture "GRSTNPBV"
    Sidedef 281 has unused lower texture "SANDSQ2"
    Sidedef 281 has unused upper texture "SANDSQ2"
    Sidedef 282 has unused upper texture "GRSTNPBV"
    Sidedef 283 has unused lower texture "SANDSQ2"
    Sidedef 283 has unused upper texture "SANDSQ2"
    Sidedef 284 has unused upper texture "GRSTNPBV"
    Sidedef 285 has unused lower texture "SANDSQ2"
    Sidedef 285 has unused upper texture "SANDSQ2"
    Sidedef 286 has unused upper texture "GRSTNPBV"
    Sidedef 287 has unused lower texture "SANDSQ2"
    Sidedef 287 has unused upper texture "SANDSQ2"
    Sidedef 288 has unused upper texture "GRSTNPBV"
    Sidedef 289 has unused lower texture "SANDSQ2"
    Sidedef 289 has unused upper texture "SANDSQ2"
    Sidedef 290 has unused upper texture "GRSTNPBV"
    Sidedef 291 has unused lower texture "SANDSQ2"
    Sidedef 291 has unused upper texture "SANDSQ2"
    Sidedef 292 has unused upper texture "GRSTNPBV"
    Sidedef 293 has unused lower texture "SANDSQ2"
    Sidedef 293 has unused upper texture "SANDSQ2"
    Sidedef 294 has unused upper texture "GRSTNPBV"
    Sidedef 295 has unused lower texture "SANDSQ2"
    Sidedef 295 has unused upper texture "SANDSQ2"
    Sidedef 296 has unused upper texture "GRSTNPBV"
    Sidedef 297 has unused lower texture "SANDSQ2"
    Sidedef 297 has unused upper texture "SANDSQ2"
    Sidedef 298 has unused upper texture "GRSTNPBV"
    Sidedef 299 has unused lower texture "SANDSQ2"
    Sidedef 299 has unused upper texture "SANDSQ2"
    Sidedef 30 has unused upper texture "SQPEB1"
    Sidedef 300 has unused upper texture "GRSTNPBV"
    Sidedef 301 has unused lower texture "SANDSQ2"
    Sidedef 301 has unused upper texture "SANDSQ2"
    Sidedef 302 has unused upper texture "GRSTNPBV"
    Sidedef 303 has unused lower texture "SANDSQ2"
    Sidedef 303 has unused upper texture "SANDSQ2"
    Sidedef 304 has unused upper texture "GRSTNPBV"
    Sidedef 305 has unused lower texture "SANDSQ2"
    Sidedef 305 has unused upper texture "SANDSQ2"
    Sidedef 306 has unused upper texture "GRSTNPBV"
    Sidedef 307 has unused lower texture "SANDSQ2"
    Sidedef 307 has unused upper texture "SANDSQ2"
    Sidedef 308 has unused upper texture "GRSTNPBV"
    Sidedef 309 has unused lower texture "SANDSQ2"
    Sidedef 309 has unused upper texture "SANDSQ2"
    Sidedef 31 has unused lower texture "SQPEB1"
    Sidedef 31 has unused upper texture "SQPEB1"
    Sidedef 310 has unused upper texture "GRSTNPBV"
    Sidedef 311 has unused lower texture "SANDSQ2"
    Sidedef 311 has unused upper texture "SANDSQ2"
    Sidedef 312 has unused upper texture "GRSTNPBV"
    Sidedef 313 has unused lower texture "SANDSQ2"
    Sidedef 313 has unused upper texture "SANDSQ2"
    Sidedef 314 has unused upper texture "GRSTNPBV"
    Sidedef 315 has unused lower texture "SANDSQ2"
    Sidedef 315 has unused upper texture "SANDSQ2"
    Sidedef 316 has unused upper texture "GRSTNPBV"
    Sidedef 317 has unused lower texture "SANDSQ2"
    Sidedef 317 has unused upper texture "SANDSQ2"
    Sidedef 318 has unused upper texture "GRSTNPBV"
    Sidedef 319 has unused lower texture "SANDSQ2"
    Sidedef 319 has unused upper texture "SANDSQ2"
    Sidedef 320 has unused upper texture "GRSTNPBV"
    Sidedef 321 has unused lower texture "SANDSQ2"
    Sidedef 321 has unused upper texture "SANDSQ2"
    Sidedef 322 has unused upper texture "GRSTNPBV"
    Sidedef 323 has unused lower texture "SANDSQ2"
    Sidedef 323 has unused upper texture "SANDSQ2"
    Sidedef 324 has unused upper texture "GRSTNPBV"
    Sidedef 325 has unused lower texture "SANDSQ2"
    Sidedef 325 has unused upper texture "SANDSQ2"
    Sidedef 326 has unused upper texture "GRSTNPBV"
    Sidedef 327 has unused lower texture "SANDSQ2"
    Sidedef 327 has unused upper texture "SANDSQ2"
    Sidedef 328 has unused upper texture "GRSTNPBV"
    Sidedef 329 has unused lower texture "SANDSQ2"
    Sidedef 329 has unused upper texture "SANDSQ2"
    Sidedef 33 has unused upper texture "SQPEB1"
    Sidedef 330 has unused upper texture "GRSTNPBV"
    Sidedef 331 has unused lower texture "SANDSQ2"
    Sidedef 331 has unused upper texture "SANDSQ2"
    Sidedef 332 has unused upper texture "GRSTNPBV"
    Sidedef 333 has unused lower texture "SANDSQ2"
    Sidedef 333 has unused upper texture "SANDSQ2"
    Sidedef 334 has unused upper texture "GRSTNPBV"
    Sidedef 335 has unused lower texture "SANDSQ2"
    Sidedef 335 has unused upper texture "SANDSQ2"
    Sidedef 336 has unused upper texture "GRSTNPBV"
    Sidedef 337 has unused lower texture "SANDSQ2"
    Sidedef 337 has unused upper texture "SANDSQ2"
    Sidedef 338 has unused upper texture "GRSTNPBV"
    Sidedef 339 has unused lower texture "SANDSQ2"
    Sidedef 339 has unused upper texture "SANDSQ2"
    Sidedef 34 has unused lower texture "SQPEB1"
    Sidedef 34 has unused upper texture "SQPEB1"
    Sidedef 340 has unused upper texture "GRSTNPBV"
    Sidedef 341 has unused lower texture "SANDSQ2"
    Sidedef 341 has unused upper texture "SANDSQ2"
    Sidedef 342 has unused upper texture "GRSTNPBV"
    Sidedef 343 has unused lower texture "SANDSQ2"
    Sidedef 343 has unused upper texture "SANDSQ2"
    Sidedef 344 has unused upper texture "GRSTNPBV"
    Sidedef 345 has unused lower texture "SANDSQ2"
    Sidedef 345 has unused upper texture "SANDSQ2"
    Sidedef 346 has unused upper texture "GRSTNPBV"
    Sidedef 347 has unused lower texture "SANDSQ2"
    Sidedef 347 has unused upper texture "SANDSQ2"
    Sidedef 348 has unused upper texture "GRSTNPBV"
    Sidedef 349 has unused lower texture "SANDSQ2"
    Sidedef 349 has unused upper texture "SANDSQ2"
    Sidedef 350 has unused upper texture "GRSTNPBV"
    Sidedef 351 has unused lower texture "SANDSQ2"
    Sidedef 351 has unused upper texture "SANDSQ2"
    Sidedef 352 has unused upper texture "GRSTNPBV"
    Sidedef 353 has unused lower texture "SANDSQ2"
    Sidedef 353 has unused upper texture "SANDSQ2"
    Sidedef 354 has unused upper texture "GRSTNPBV"
    Sidedef 355 has unused lower texture "SANDSQ2"
    Sidedef 355 has unused upper texture "SANDSQ2"
    Sidedef 356 has unused upper texture "GRSTNPBV"
    Sidedef 357 has unused lower texture "SANDSQ2"
    Sidedef 357 has unused upper texture "SANDSQ2"
    Sidedef 358 has unused upper texture "GRSTNPBV"
    Sidedef 359 has unused lower texture "SANDSQ2"
    Sidedef 359 has unused upper texture "SANDSQ2"
    Sidedef 36 has unused lower texture "SANDSQ2"
    Sidedef 36 has unused upper texture "SANDSQ2"
    Sidedef 360 has unused upper texture "GRSKULL1"
    Sidedef 361 has unused lower texture "SANDSQ2"
    Sidedef 361 has unused upper texture "SANDSQ2"
    Sidedef 362 has unused upper texture "GRSKULL1"
    Sidedef 363 has unused lower texture "SANDSQ2"
    Sidedef 363 has unused upper texture "SANDSQ2"
    Sidedef 364 has unused upper texture "GRSKULL1"
    Sidedef 365 has unused lower texture "SANDSQ2"
    Sidedef 365 has unused upper texture "SANDSQ2"
    Sidedef 366 has unused upper texture "GRSKULL1"
    Sidedef 367 has unused lower texture "SANDSQ2"
    Sidedef 367 has unused upper texture "SANDSQ2"
    Sidedef 387 has unused lower texture "CSTLRCK"
    Sidedef 387 has unused upper texture "CSTLRCK"
    Sidedef 39 has unused lower texture "SANDSQ2"
    Sidedef 39 has unused upper texture "SANDSQ2"
    Sidedef 391 has unused upper texture "CSTLRCK"
    Sidedef 392 has unused lower texture "CSTLRCK"
    Sidedef 392 has unused upper texture "CSTLRCK"
    Sidedef 396 has unused lower texture "SANDSQ2"
    Sidedef 396 has unused upper texture "SANDSQ2"
    Sidedef 397 has unused lower texture "SANDSQ2"
    Sidedef 397 has unused upper texture "SANDSQ2"
    Sidedef 399 has unused lower texture "SANDSQ2"
    Sidedef 399 has unused upper texture "SANDSQ2"
    Sidedef 400 has unused lower texture "SANDSQ2"
    Sidedef 400 has unused upper texture "SANDSQ2"
    Sidedef 401 has unused lower texture "SANDSQ2"
    Sidedef 401 has unused upper texture "SANDSQ2"
    Sidedef 402 has unused lower texture "SANDSQ2"
    Sidedef 402 has unused upper texture "SANDSQ2"
    Sidedef 403 has unused lower texture "SANDSQ2"
    Sidedef 403 has unused upper texture "SANDSQ2"
    Sidedef 404 has unused lower texture "SANDSQ2"
    Sidedef 404 has unused upper texture "SANDSQ2"
    Sidedef 405 has unused lower texture "SANDSQ2"
    Sidedef 405 has unused upper texture "SANDSQ2"
    Sidedef 406 has unused lower texture "SANDSQ2"
    Sidedef 406 has unused upper texture "SANDSQ2"
    Sidedef 408 has unused lower texture "SANDSQ2"
    Sidedef 408 has unused upper texture "SANDSQ2"
    Sidedef 409 has unused lower texture "SANDSQ2"
    Sidedef 409 has unused upper texture "SANDSQ2"
    Sidedef 41 has unused lower texture "SQPEB1"
    Sidedef 41 has unused upper texture "SQPEB1"
    Sidedef 410 has unused lower texture "SANDSQ2"
    Sidedef 410 has unused upper texture "SANDSQ2"
    Sidedef 412 has unused lower texture "SANDSQ2"
    Sidedef 412 has unused upper texture "SANDSQ2"
    Sidedef 413 has unused lower texture "SANDSQ2"
    Sidedef 413 has unused upper texture "SANDSQ2"
    Sidedef 414 has unused lower texture "SANDSQ2"
    Sidedef 414 has unused upper texture "SANDSQ2"
    Sidedef 415 has unused lower texture "SANDSQ2"
    Sidedef 415 has unused upper texture "SANDSQ2"
    Sidedef 416 has unused lower texture "SANDSQ2"
    Sidedef 416 has unused upper texture "SANDSQ2"
    Sidedef 418 has unused lower texture "SANDSQ2"
    Sidedef 418 has unused upper texture "SANDSQ2"
    Sidedef 419 has unused lower texture "SANDSQ2"
    Sidedef 419 has unused upper texture "SANDSQ2"
    Sidedef 42 has unused lower texture "SQPEB1"
    Sidedef 42 has unused upper texture "SQPEB1"
    Sidedef 422 has unused lower texture "SQPEB1"
    Sidedef 422 has unused upper texture "SQPEB1"
    Sidedef 423 has unused lower texture "SQPEB1"
    Sidedef 423 has unused upper texture "SQPEB1"
    Sidedef 427 has unused upper texture "SQPEB1"
    Sidedef 428 has unused lower texture "SQPEB1"
    Sidedef 428 has unused upper texture "SQPEB1"
    Sidedef 434 has unused upper texture "SQPEB1"
    Sidedef 435 has unused lower texture "SQPEB1"
    Sidedef 435 has unused upper texture "SQPEB1"
    Sidedef 44 has unused lower texture "SANDSQ2"
    Sidedef 44 has unused upper texture "SANDSQ2"
    Sidedef 444 has unused lower texture "SANDSQ2"
    Sidedef 444 has unused upper texture "SANDSQ2"
    Sidedef 445 has unused lower texture "SANDSQ2"
    Sidedef 445 has unused upper texture "SANDSQ2"
    Sidedef 447 has unused lower texture "SANDSQ2"
    Sidedef 447 has unused upper texture "SANDSQ2"
    Sidedef 448 has unused lower texture "SANDSQ2"
    Sidedef 448 has unused upper texture "SANDSQ2"
    Sidedef 474 has unused lower texture "SANDSQ2"
    Sidedef 474 has unused upper texture "SANDSQ2"
    Sidedef 475 has unused lower texture "SANDSQ2"
    Sidedef 475 has unused upper texture "SANDSQ2"
    Sidedef 477 has unused lower texture "SANDSQ2"
    Sidedef 477 has unused upper texture "SANDSQ2"
    Sidedef 478 has unused lower texture "SANDSQ2"
    Sidedef 478 has unused upper texture "SANDSQ2"
    Sidedef 480 has unused lower texture "SANDSQ2"
    Sidedef 480 has unused upper texture "SANDSQ2"
    Sidedef 481 has unused lower texture "SANDSQ2"
    Sidedef 481 has unused upper texture "SANDSQ2"
    Sidedef 483 has unused lower texture "SANDSQ2"
    Sidedef 483 has unused upper texture "SANDSQ2"
    Sidedef 484 has unused lower texture "SANDSQ2"
    Sidedef 484 has unused upper texture "SANDSQ2"
    Sidedef 49 has unused lower texture "SANDSQ2"
    Sidedef 49 has unused upper texture "SANDSQ2"
    Sidedef 61 has unused lower texture "CSTLRCK"
    Sidedef 61 has unused upper texture "CSTLRCK"
    Sidedef 65 has unused lower texture "RCKSNMUD"
    Sidedef 65 has unused upper texture "RCKSNMUD"
    Sidedef 69 has unused lower texture "CSTLRCK"
    Sidedef 69 has unused upper texture "CSTLRCK"
    Sidedef 7 has unused upper texture "SQPEB1"
    Sidedef 73 has unused lower texture "CSTLRCK"
    Sidedef 73 has unused upper texture "CSTLRCK"
    Sidedef 79 has unused lower texture "RCKSNMUD"
    Sidedef 79 has unused upper texture "RCKSNMUD"
    Sidedef 8 has unused lower texture "SQPEB1"
    Sidedef 8 has unused upper texture "SQPEB1"
    Sidedef 83 has unused lower texture "CSTLRCK"
    Sidedef 83 has unused upper texture "CSTLRCK"
    Sidedef 87 has unused lower texture "CSTLRCK"
    Sidedef 87 has unused upper texture "CSTLRCK"
    Sidedef 91 has unused lower texture "CSTLRCK"
    Sidedef 91 has unused upper texture "CSTLRCK"
    Sidedef 95 has unused lower texture "SANDSQ2"
    Sidedef 95 has unused upper texture "SANDSQ2"
    Sidedef 97 has unused lower texture "SANDSQ2"
    Sidedef 97 has unused upper texture "SANDSQ2"
    Thing 1 (Wall Torch) is stuck in linedef 111 at -32, 536
    Thing 2 (Wall Torch) is stuck in linedef 110 at 8, 480
    Thing 5 (Wall Torch) is stuck in linedef 351 at -200, 1792
    Thing 6 (Wall Torch) is stuck in linedef 349 at -200, 1536
    Vertex 11 overlaps line 189 without splitting it
    Vertex 149 overlaps line 138 without splitting it
    Vertex 25 overlaps line 22 without splitting it
    Vertex 25 overlaps line 91 without splitting it
    Vertex 26 overlaps line 21 without splitting it
    Vertex 4 overlaps line 298 without splitting it
    Vertex 4 overlaps line 4 without splitting it
    Vertex 5 overlaps line 57 without splitting it
    Vertices 11 and 149 have the same position
    Vertices 25 and 26 have the same position
    Vertices 4 and 5 have the same position
    

Share this post


Link to post
andrewj said:

This map has 3 vertex pairs that are not connected (i.e. sitting on top of each other).

Shouldn't they be automatically merged if less than 1 unit apart? :) Like in Doom Builder.

Share this post


Link to post

In normal line drawing, a highlighted vertex will be used as a destination (so duplicated vertices at a point can never happen).

But when dragging stuff around or copy/paste, there is no automatic splitting or merging.

Share this post


Link to post
andrewj said:

But when dragging stuff around or copy/paste, there is no automatic splitting or merging.

So if I drag a vertex to merge a linedef into another (for instance to reduce or remove altogether an inner sector adjacent to a wall), the linedefs and vertices won't merge?

Scenario I'm talking about :

A
x----------------x
| SECTOR 1      /.
|     x--------x .
|    / B         .
|   /            .
|  /             .
| /    SECTOR 2  .
|/               .
x-----------------

MERGE VERTEX B INTO VERTEX A.

Share this post


Link to post

Dragging a single vertex onto another will merge them, but I just tried that scenario and it creates an unclosed sector, because when it merges the two linedefs at A and B it doesn't know which sectors it should keep on the merged line.

I have wanted to better handle cases liket his, or dragging complex geometry (multiple vertices) onto nearby geometry, but it is very non-trivial to actually do it.

Share this post


Link to post

Okay, so there are two linedefs merging. You know which vertices are going to join which, so you also know the linedef orientations. From this, you know which sectors are on which sides, and which sides are inner and outer. On merging, the inner sides will disappear. What will remain will be the outer sides. So the resulting linedef should have, as its two sides, the outer sides of the lines to be merged.

side 1
x--------x
side 2                      side 1
                   =====>.  x--------x
side 3                      side 4
x--------x
side 4
Is this feasible?

Share this post


Link to post

Yes, for two linedefs which share one vertex, I'm 99% sure that is the correct logic.

Added to the TODO....

Share this post


Link to post

I realized you also need to know the relative positions of the lines to know which sides are "outer" and "inner". In the above picture, if I swapped the two lines, the final line would have sides 2 and 3, not 1 and 4.

I believe that two merging linedefs must always have one sector in common, or the void in common. The sides which have the same sector, or otherwise the two null sides, are going to be removed, and the resulting linedef shall have the other, non-common sides.

Share this post


Link to post

Yeah.

BTW, the code already has a MergeConnectedLines() function to handle this case. Just need to get it to handle the sidedefs properly.

Share this post


Link to post

Create an account or sign in to comment

You need to be a member in order to leave a comment

Create an account

Sign up for a new account in our community. It's easy!

Register a new account

Sign in

Already have an account? Sign in here.

Sign In Now
×