| View previous topic :: View next topic |
| Author |
Message |
Javier López
Site Admin
Joined: 26 Dec 2007
Posts: 515
Location: Murcia, Spain
|
 Posted: Tue Apr 22, 2008 2:55 pm Post subject: IndieLib 1.01 - Report bugs here! |
 |
|
Here it is!
[Edit]Removed: old release[/Edit]
Fixed al know bugs reported, thanks to all you!
If you find new ones, report theme here.
Last edited by Javier López on Sun Dec 21, 2008 2:16 pm; edited 1 time in total |
|
| Back to top |
|
 |
ZCCdark203
Joined: 14 Apr 2008
Posts: 9
|
| Posted: Tue May 06, 2008 2:04 pm Post subject: |
|
|
In "\documentation\html\group___graphical___objects.html" you've made a mistake in the documentation of IND_Render::BlitWrapSurface. The function itself displays 5 parameters, but in the following clarification 7 parameters are displayed.
In the documentation of the load and save functions of both IND_SurfaceManager and IND_ImageManager stands this (or something similar):
"y" should be replaced by "and". This is no bug, though. |
|
| Back to top |
|
|
krajek
Joined: 13 Jun 2008
Posts: 11
|
| Posted: Fri Jun 13, 2008 9:45 pm Post subject: |
|
|
I have played with IndieLib for a while and here are my conclusions/tips
*Method GetPozZ is still there.
*IMO methods like SetPosX and SetAngleX should be added. Simply moving right one pixel now means :
| Code: |
entity.SetPosition(entity.GetPosX()+1,entity.GetPosY(),entity.GetPozZ());
|
*Consider using unions for private data which is not shared between various entity types in your entity2d class(I haven't tested entity3d yet). It would save a lot of memory without damaging performance.
*It's too hard to find who's the author of the library and the only place where I found e-mail to you, were header files.
Keep up the good work, your engine has great potencial. |
|
| Back to top |
|
|
Javier López
Site Admin
Joined: 26 Dec 2007
Posts: 515
Location: Murcia, Spain
|
| Posted: Tue Jul 15, 2008 11:11 am Post subject: |
|
|
| Thank you very much Krajek! |
|
| Back to top |
|
|
krajek
Joined: 13 Jun 2008
Posts: 11
|
| Posted: Thu Jul 17, 2008 1:47 pm Post subject: |
|
|
Here are next (really)small problems. Maybe they are not but they are small improvements.
1.
Here's some of my code :
| Code: |
bool success = CIndieLib::Instance()->SurfaceManager->Add(
pSurface, //NewSurface
const_cast<char*>(data.mFilePath.c_str()), //Path <--- dirty cast
data.mType, //Type
data.mQuality //Quality
);
|
'Add' method should obviously take const char * as 'Path' parameter, so no one would be forced to used dirty const_cast. The same is with methods :
bool IND_Entity2d::SetBoundingAreas ( char * pFile )
void IND_Entity2d::SetText ( char * pText )
and some others.
2.
There is a lack of SetVisible(bool) and GetVisible(bool) methods in IND_Entity2d.
3.
Some kind of resource manager should be added to the library. Currently i'm writing one, and I will show you it when i'll finish. Maybe you will even consider adding it to next IndieLib releases  .
Same as previously, keep up good work, good luck.
BTW. I'm Polish;). |
|
| Back to top |
|
|
Javier López
Site Admin
Joined: 26 Dec 2007
Posts: 515
Location: Murcia, Spain
|
| Posted: Fri Jul 18, 2008 11:51 am Post subject: |
|
|
| Polish people never stop of surprising me. You are right in everything Krajek. I will add this improvements to the list. |
|
| Back to top |
|
|
krajek
Joined: 13 Jun 2008
Posts: 11
|
| Posted: Sun Jul 20, 2008 6:43 pm Post subject: |
|
|
And it's me again  .
Let's look at the
| Code: |
void IND_Entity2dManager::End().
|
documentation. It says :
| Quote: |
This function frees the manager and all the objects that it contains.
|
However if I don't release manually even one IND_Entity2d with method Delete(), then calling method End() crashes with AV. IMO End() should handle such situations gracefully and auto-delete existing entities.
BTW : I believe one should use term 'method' instead of 'function' in class documentation, but i'm not sure. |
|
| Back to top |
|
|
krajek
Joined: 13 Jun 2008
Posts: 11
|
| Posted: Mon Aug 25, 2008 11:54 am Post subject: |
|
|
Lately I had to use 'blit directly' part of your lib. And there are problem, because parameters Width and Height of SetTransform2d are meaningless. It doesn't matter whether you pass 0,128 or 1000000 as a Width or Height, result is the same. To change size one has to use parameters scaleX ang scaleY.
EDIT:
In tutorial projects maybe it will be good to change working directory to :
'../../../../exe/vc2008/' so it will be possible to simply press F5 and run it. No you can't do this because dll is missing. |
|
| Back to top |
|
|
Javier López
Site Admin
Joined: 26 Dec 2007
Posts: 515
Location: Murcia, Spain
|
| Posted: Wed Sep 24, 2008 4:27 pm Post subject: |
|
|
I have to apologize for the BIG delay in my answer.
pWidth and pHeight in SetTransform2d are not the size of the image for scaling it. In fact HAS TO BE the width and height of the image. They are used for the Hot Spot and Mirroring.
In order to change the size, as you said, you should use pScaleX and pScaleY.
I will comment this better both in the tutorials and in the Api Reference. Sorry!
| Quote: | In tutorial projects maybe it will be good to change working directory to :
'../../../../exe/vc2008/' so it will be possible to simply press F5 and run it. No you can't do this because dll is missing. |
Good idea! |
|
| Back to top |
|
|
|
FAQ
Search
Memberlist
Usergroups
Register
Profile
Log in to check your private messages
Log in
