Jump to content

Recommended Posts

Posted

Yes, I have a ridiculous number of entities. More than 50. I lost count.

It's the OreSpawn mod. Tons and tons of critters...

And yet, I'm trying to add a few more! :)

 

PROBLEM: EntityID 128 does not work. It crashes.

Everything below that is fine. Anything 128 and up, goes splat.

 

From my log:

2013-11-03 09:36:40 [iNFO] [sTDOUT] Small Worm ID = 124

2013-11-03 09:36:40 [iNFO] [sTDOUT] Cloud Shark ID = 128

 

Worm and Cassowary(127) are fine.

Cloud Shark and Gold Fish(129) both crash.

 

From the Minecraft log:

 

---- Minecraft Crash Report ----

// There are four lights!

 

Time: 11/3/13 9:48 AM

Description: Unexpected error

 

java.lang.NullPointerException

at net.minecraft.client.multiplayer.NetClientHandler.handleMobSpawn(NetClientHandler.java:952)

at net.minecraft.network.packet.Packet24MobSpawn.processPacket(Packet24MobSpawn.java:143)

at net.minecraft.network.MemoryConnection.processReadPackets(MemoryConnection.java:89)

at net.minecraft.client.Minecraft.runTick(Minecraft.java:1966)

at net.minecraft.client.Minecraft.runGameLoop(Minecraft.java:909)

at net.minecraft.client.Minecraft.run(Minecraft.java:837)

at net.minecraft.client.main.Main.main(Main.java:93)

at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)

at sun.reflect.NativeMethodAccessorImpl.invoke(Unknown Source)

at sun.reflect.DelegatingMethodAccessorImpl.invoke(Unknown Source)

at java.lang.reflect.Method.invoke(Unknown Source)

at net.minecraft.launchwrapper.Launch.launch(Launch.java:131)

at net.minecraft.launchwrapper.Launch.main(Launch.java:27)

 

From the Forge log:

 

(nothing)

 

 

 

It crashes after trying to create the entity by ID, failing, returning NULL, and then accessing NULL.xposition.

 

Seems to be a bug with the entity map, which appears to be 255 bits,

but is apparently only useful up to 127. Looks like a signed/unsigned byte problem.

Here's a thought.... make it an INT like it should be, and bump the map up to 1024 in size.

That would be nice!

 

In the meantime, IS THERE A WORKAROUND???

 

 

 

Odds are good I've been programming since before you were born. Yeah. I'm OLD school.

Posted

Still happens in Forge 942.

Here is a bit more info...

 

From NetClientHandler.java:

 

        System.out.printf("Spawning entity %d\n", par1Packet24MobSpawn.type );

        EntityLivingBase entitylivingbase = (EntityLivingBase)EntityList.createEntityByID(par1Packet24MobSpawn.type, this.mc.theWorld);

        entitylivingbase.serverPosX = par1Packet24MobSpawn.xPosition; (THIS IS WHERE THE NULL ptr CRASH occurs)

 

 

produces "Spawning entity -128"

 

(the ID returned from findGlobalUniqueEntityId() was +128!!! How did it end up negative across the net???)

 

Adding the line:

 

if(par1Packet24MobSpawn.type < 0)par1Packet24MobSpawn.type = 256+par1Packet24MobSpawn.type;

 

between the printf and the createEntityByID() works perfectly, but I don't know where else this

problem might occur, and what other functions may be affected. It also violates Forge code, so I can't

use this fix in the actual distributable mod... :(

 

PLEASE fix...

 

 

Odds are good I've been programming since before you were born. Yeah. I'm OLD school.

Posted

Except that any number I pick over 127 is going to end up negative when sent over the network interface.

findGlobalUniqueEntityId() returns the correct value of 128.

par1Packet24MobSpawn.type is wrong, at -128.

So, somewhere between when I get a correct positive 128, and send it accross the net,

and the other side tries to spawn it, it becomes a negative 128.

I have no way to bypass all that crap.

In other words, even if I picked arbitrary numbers myself,

it would still fuck it up.

 

It's a bug.

Please fix...

 

Odds are good I've been programming since before you were born. Yeah. I'm OLD school.

Posted

It's not a bug, it's how minecraft works and most likely Lex/cpw's most likely not going to change it.

Read the EAQ before posting! OR ELSE!

 

This isn't building better software, its trying to grab a place in the commit list of a highly visible github project.

 

www.forgeessentials.com

 

Don't PM me, I don't check this account unless I have to.

Posted

Lol! Actually, I discovered that more than 30 years ago.

Seems the folks at Minecraft still haven't...

 

It's actually a bug in their code, as they expect the compiler to do things in a different order

than it actually does them... The offending line, from Packet24MobSpawn.java:

  this.type = par1DataInput.readByte() & 255;

The compiler reads the byte, ANDs it with 255, and THEN converts it to an integer.

Obviously they intended it the other way around.

 

On a side note, yes, I had filed a bug report with minecraft detailing the problem,

and they did in fact blow it off. "Resolved" it as "incomplete".

How it could be incomplete I don't know... I even gave them the fixed code!

Sheesh...

 

So...

 

Is there a way to globally override the Packet24MobSpawn class at runtime when

Forge loads my mod???

 

Or perhaps MCP could patch it???

 

Or you all could patch it??? PLEASE???

 

I need standard spawning to work properly... Are there any other options???

 

I desperately need space for moar mobs!

:)

 

Odds are good I've been programming since before you were born. Yeah. I'm OLD school.

Posted

What you SHOULD do is move to the normal per-mod spawning mechanics. Which uses a modid + mobid combo to allow you have compleetly seperate ID set and not cause any conflicts, overflow, etc..

We will not "fix" this as you're doing it wrong.

See the EntityRegistry in FML.

  • Like 1

I do Forge for free, however the servers to run it arn't free, so anything is appreciated.
Consider supporting the team on Patreon

Posted

Y'all are almost as bad as Mojang... Yep. They refuse to fix their bugs too.

I looked at EntityRegistry. It is NOT self explanatory.

It would appear, from testing and trial and error (because I could NOT find

an example of the proper way to create/spawn mobs anywhere!!!!),

that this:

 

GoldFishID = EntityRegistry.findGlobalUniqueEntityId();

EntityRegistry.registerGlobalEntityID(GoldFish.class, "GoldFish", GoldFishID, 0xddffdd, 0x669955);

EntityRegistry.registerModEntity(GoldFish.class, "GoldFish", GoldFishID, this, 64, 1, false);

 

is at least one way that appears to function.

(It doesn't crash, at least!)

 

IS THIS CORRECT?

If not, please point me at an example or two or three or four... I could not find any!

 

I would like to know for sure what the CORRECT way is,

because I've got a shitload of code that I will have to update,

and I really don't want to do that more than once.

 

Thanks!

 

Odds are good I've been programming since before you were born. Yeah. I'm OLD school.

Posted

    /**
     * Register the mod entity type with FML

     * @param entityClass The entity class
     * @param entityName A unique name for the entity
     * @param id A mod specific ID for the entity
     * @param mod The mod
     * @param trackingRange The range at which MC will send tracking updates
     * @param updateFrequency The frequency of tracking updates
     * @param sendsVelocityUpdates Whether to send velocity information packets as well
     */
    public static void registerModEntity(Class<? extends Entity> entityClass, String entityName, int id, Object mod, int trackingRange, int updateFrequency, boolean sendsVelocityUpdates)
    {
        instance().doModEntityRegistration(entityClass, entityName, id, mod, trackingRange, updateFrequency, sendsVelocityUpdates);
    }

Humm humm yes that is SOOOOOO FUCKING CONFUSING!!!!!

Note what it says about the id:MOD SPECIFIC ID

Cuz you know.. id conflicts between mods are stupid and shouldnt happen. Hence Mod specific...

 

Either way, don't be a fucking cunt. The id issue isnt OUR issue. It's a Minecraft issue that we have had resolved for ages.

The ONLY reason the global ID BS is there is for MODLOADER compatibility and it's going aweay in 1.7

  • Like 1

I do Forge for free, however the servers to run it arn't free, so anything is appreciated.
Consider supporting the team on Patreon

Posted

Lol!

Except that that doesn't work!

It ONLY works if you include registering your global ID. IE:

 

LurkingTerrorID = 182;

EntityRegistry.registerGlobalEntityID(LurkingTerror.class, "Lurking Terror", LurkingTerrorID);

EntityRegistry.registerModEntity(LurkingTerror.class, "Lurking Terror", LurkingTerrorID, this, 64, 1, false);

 

Try learning your own code.

Sheesh.

 

And you expect me to sort through all this spaghetti and try to figure out which is old moldy crap,

and which is the "new and improved" way of doing things???

Yeah, sure, I'll just read the NON-EXISTENT comments...

 

Oh well. Thanks anyway.

I've figured it out and can carry on with life and making a bunch of kids happy again.

No thanks to you!

Pbbbt!

 

Try being a little less of an ass next time.

There is really no need for it.

 

Odds are good I've been programming since before you were born. Yeah. I'm OLD school.

Posted

Ya I dont know what the fuck you're doing wrong but entities do NOT need a global registration. As things are picked up for you already when you do the mod registration as long as you DONT fuck it up by doing the global registry.

 

But ya, banned cuz you're just being a cunt. The class is fairly fine if you read it. And as evidenced by the code snippit I pasted comments DO exist.

 

 

Anyways have fun, be less of a cunt, enjoy modding!

  • Like 1

I do Forge for free, however the servers to run it arn't free, so anything is appreciated.
Consider supporting the team on Patreon

Guest
This topic is now closed to further replies.

Announcements



  • Recently Browsing

    • No registered users viewing this page.
  • Posts

    • Version 1.19 - Forge 41.0.63 I want to create a wolf entity that I can ride, so far it seems to be working, but the problem is that when I get on the wolf, I can’t control it. I then discovered that the issue is that the server doesn’t detect that I’m riding the wolf, so I’m struggling with synchronization. However, it seems to not be working properly. As I understand it, the server receives the packet but doesn’t register it correctly. I’m a bit new to Java, and I’ll try to provide all the relevant code and prints *The comments and prints are translated by chatgpt since they were originally in Spanish* Thank you very much in advance No player is mounted, or the passenger is not a player. No player is mounted, or the passenger is not a player. No player is mounted, or the passenger is not a player. No player is mounted, or the passenger is not a player. No player is mounted, or the passenger is not a player. MountableWolfEntity package com.vals.valscraft.entity; import com.vals.valscraft.network.MountSyncPacket; import com.vals.valscraft.network.NetworkHandler; import net.minecraft.client.Minecraft; import net.minecraft.network.syncher.EntityDataAccessor; import net.minecraft.network.syncher.EntityDataSerializers; import net.minecraft.network.syncher.SynchedEntityData; import net.minecraft.server.MinecraftServer; import net.minecraft.server.level.ServerPlayer; import net.minecraft.world.entity.EntityType; import net.minecraft.world.entity.Mob; import net.minecraft.world.entity.ai.attributes.AttributeSupplier; import net.minecraft.world.entity.ai.attributes.Attributes; import net.minecraft.world.entity.animal.Wolf; import net.minecraft.world.entity.player.Player; import net.minecraft.world.entity.Entity; import net.minecraft.world.InteractionHand; import net.minecraft.world.InteractionResult; import net.minecraft.world.item.ItemStack; import net.minecraft.world.item.Items; import net.minecraft.world.level.Level; import net.minecraft.world.phys.Vec3; import net.minecraftforge.event.TickEvent; import net.minecraftforge.eventbus.api.SubscribeEvent; import net.minecraftforge.network.PacketDistributor; public class MountableWolfEntity extends Wolf { private boolean hasSaddle; private static final EntityDataAccessor<Byte> DATA_ID_FLAGS = SynchedEntityData.defineId(MountableWolfEntity.class, EntityDataSerializers.BYTE); public MountableWolfEntity(EntityType<? extends Wolf> type, Level level) { super(type, level); this.hasSaddle = false; } @Override protected void defineSynchedData() { super.defineSynchedData(); this.entityData.define(DATA_ID_FLAGS, (byte)0); } public static AttributeSupplier.Builder createAttributes() { return Wolf.createAttributes() .add(Attributes.MAX_HEALTH, 20.0) .add(Attributes.MOVEMENT_SPEED, 0.3); } @Override public InteractionResult mobInteract(Player player, InteractionHand hand) { ItemStack itemstack = player.getItemInHand(hand); if (itemstack.getItem() == Items.SADDLE && !this.hasSaddle()) { if (!player.isCreative()) { itemstack.shrink(1); } this.setSaddle(true); return InteractionResult.SUCCESS; } else if (!level.isClientSide && this.hasSaddle()) { player.startRiding(this); MountSyncPacket packet = new MountSyncPacket(true); // 'true' means the player is mounted NetworkHandler.CHANNEL.sendToServer(packet); // Ensure the server handles the packet return InteractionResult.SUCCESS; } return InteractionResult.PASS; } @Override public void travel(Vec3 travelVector) { if (this.isVehicle() && this.getControllingPassenger() instanceof Player) { System.out.println("The wolf has a passenger."); System.out.println("The passenger is a player."); Player player = (Player) this.getControllingPassenger(); // Ensure the player is the controller this.setYRot(player.getYRot()); this.yRotO = this.getYRot(); this.setXRot(player.getXRot() * 0.5F); this.setRot(this.getYRot(), this.getXRot()); this.yBodyRot = this.getYRot(); this.yHeadRot = this.yBodyRot; float forward = player.zza; float strafe = player.xxa; if (forward <= 0.0F) { forward *= 0.25F; } this.flyingSpeed = this.getSpeed() * 0.1F; this.setSpeed((float) this.getAttributeValue(Attributes.MOVEMENT_SPEED) * 1.5F); this.setDeltaMovement(new Vec3(strafe, travelVector.y, forward).scale(this.getSpeed())); this.calculateEntityAnimation(this, false); } else { // The wolf does not have a passenger or the passenger is not a player System.out.println("No player is mounted, or the passenger is not a player."); super.travel(travelVector); } } public boolean hasSaddle() { return this.hasSaddle; } public void setSaddle(boolean hasSaddle) { this.hasSaddle = hasSaddle; } @Override protected void dropEquipment() { super.dropEquipment(); if (this.hasSaddle()) { this.spawnAtLocation(Items.SADDLE); this.setSaddle(false); } } @SubscribeEvent public static void onServerTick(TickEvent.ServerTickEvent event) { if (event.phase == TickEvent.Phase.START) { MinecraftServer server = net.minecraftforge.server.ServerLifecycleHooks.getCurrentServer(); if (server != null) { for (ServerPlayer player : server.getPlayerList().getPlayers()) { if (player.isPassenger() && player.getVehicle() instanceof MountableWolfEntity) { MountableWolfEntity wolf = (MountableWolfEntity) player.getVehicle(); System.out.println("Tick: " + player.getName().getString() + " is correctly mounted on " + wolf); } } } } } private boolean lastMountedState = false; @Override public void tick() { super.tick(); if (!this.level.isClientSide) { // Only on the server boolean isMounted = this.isVehicle() && this.getControllingPassenger() instanceof Player; // Only print if the state changed if (isMounted != lastMountedState) { if (isMounted) { Player player = (Player) this.getControllingPassenger(); // Verify the passenger is a player System.out.println("Server: Player " + player.getName().getString() + " is now mounted."); } else { System.out.println("Server: The wolf no longer has a passenger."); } lastMountedState = isMounted; } } } @Override public void addPassenger(Entity passenger) { super.addPassenger(passenger); if (passenger instanceof Player) { Player player = (Player) passenger; if (!this.level.isClientSide && player instanceof ServerPlayer) { // Send the packet to the server to indicate the player is mounted NetworkHandler.CHANNEL.send(PacketDistributor.PLAYER.with(() -> (ServerPlayer) player), new MountSyncPacket(true)); } } } @Override public void removePassenger(Entity passenger) { super.removePassenger(passenger); if (passenger instanceof Player) { Player player = (Player) passenger; if (!this.level.isClientSide && player instanceof ServerPlayer) { // Send the packet to the server to indicate the player is no longer mounted NetworkHandler.CHANNEL.send(PacketDistributor.PLAYER.with(() -> (ServerPlayer) player), new MountSyncPacket(false)); } } } @Override public boolean isControlledByLocalInstance() { Entity entity = this.getControllingPassenger(); return entity instanceof Player; } @Override public void positionRider(Entity passenger) { if (this.hasPassenger(passenger)) { double xOffset = Math.cos(Math.toRadians(this.getYRot() + 90)) * 0.4; double zOffset = Math.sin(Math.toRadians(this.getYRot() + 90)) * 0.4; passenger.setPos(this.getX() + xOffset, this.getY() + this.getPassengersRidingOffset() + passenger.getMyRidingOffset(), this.getZ() + zOffset); } } } MountSyncPacket package com.vals.valscraft.network; import com.vals.valscraft.entity.MountableWolfEntity; import net.minecraft.network.FriendlyByteBuf; import net.minecraft.server.level.ServerLevel; import net.minecraft.server.level.ServerPlayer; import net.minecraft.world.entity.Entity; import net.minecraft.world.entity.player.Player; import net.minecraftforge.network.NetworkEvent; import java.util.function.Supplier; public class MountSyncPacket { private final boolean isMounted; public MountSyncPacket(boolean isMounted) { this.isMounted = isMounted; } public void encode(FriendlyByteBuf buffer) { buffer.writeBoolean(isMounted); } public static MountSyncPacket decode(FriendlyByteBuf buffer) { return new MountSyncPacket(buffer.readBoolean()); } public void handle(NetworkEvent.Context context) { context.enqueueWork(() -> { ServerPlayer player = context.getSender(); // Get the player from the context if (player != null) { // Verifies if the player has dismounted if (!isMounted) { Entity vehicle = player.getVehicle(); if (vehicle instanceof MountableWolfEntity wolf) { // Logic to remove the player as a passenger wolf.removePassenger(player); System.out.println("Server: Player " + player.getName().getString() + " is no longer mounted."); } } } }); context.setPacketHandled(true); // Marks the packet as handled } } networkHandler package com.vals.valscraft.network; import com.vals.valscraft.valscraft; import net.minecraft.resources.ResourceLocation; import net.minecraftforge.network.NetworkRegistry; import net.minecraftforge.network.simple.SimpleChannel; import net.minecraftforge.network.NetworkEvent; import java.util.function.Supplier; public class NetworkHandler { private static final String PROTOCOL_VERSION = "1"; public static final SimpleChannel CHANNEL = NetworkRegistry.newSimpleChannel( new ResourceLocation(valscraft.MODID, "main"), () -> PROTOCOL_VERSION, PROTOCOL_VERSION::equals, PROTOCOL_VERSION::equals ); public static void init() { int packetId = 0; // Register the mount synchronization packet CHANNEL.registerMessage( packetId++, MountSyncPacket.class, MountSyncPacket::encode, MountSyncPacket::decode, (msg, context) -> msg.handle(context.get()) // Get the context with context.get() ); } }  
    • Do you use features of inventory profiles next (ipnext) or is there a change without it?
    • Remove rubidium - you are already using embeddium, which is a fork of rubidium
  • Topics

×
×
  • Create New...

Important Information

By using this site, you agree to our Terms of Use.