Skip to content

Conversation

@obseletecode72
Copy link

Some servers even when you send the viewDistance setting to it, they can send the default chunk amount that has been set, so you must check how much chunks you are receiving before saving them in to memory

Some servers even when you send the viewDistance setting to it, they can send the default chunk amount that has been set, so you must check how much chunks you are receiving before saving them in to memory
@rom1504
Copy link
Member

rom1504 commented Oct 22, 2025

Please undo irrelevant changes and fix lint

@obseletecode72
Copy link
Author

be more specific please.

Copy link
Member

@rom1504 rom1504 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Undo all of those and many similar unnecessary changes to spaces

const assert = require('assert')
const Painting = require('../painting')
const { onceWithCleanup } = require('../promise_utils')

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Undo

}

function inject (bot, { version, storageBuilder, hideErrors }) {
function inject(bot, { version, storageBuilder, hideErrors }) {
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Undo

const paintingsById = {}

function addPainting (painting) {
function addPainting(painting) {
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Undo

}

function deletePainting (painting) {
function deletePainting(painting) {
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Undo

}

function delColumn (chunkX, chunkZ) {
function delColumn(chunkX, chunkZ) {
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Undo

if (dist > viewDist) return

if (!args.bitMap && args.groundUp) {
// stop storing the chunk column
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Undo

}
let column = bot.world.getColumn(args.x, args.z)
if (!column) {
// Allocates new chunk object while taking world's custom min/max height into account
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Undo

if (args.biomes !== undefined) {
column.loadBiomes(args.biomes)
}
if (args.biomes !== undefined) column.loadBiomes(args.biomes)
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Undo

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants