* implemented CubeMaps
[glBitmap.git] / glBitmap.pas
index debcd36..513a9e6 100644 (file)
@@ -4,7 +4,6 @@ http://www.opengl24.de/index.php?cat=header&file=glbitmap
 
 modified by Delphi OpenGL Community (http://delphigl.com/)
 
-
 ------------------------------------------------------------
 The contents of this file are used with permission, subject to
 the Mozilla Public License Version 1.1 (the "License"); you may
@@ -244,6 +243,7 @@ unit glBitmap;
 {$DEFINE GLB_LAZARUS}
 
 
+
 // activate to enable the support of SDL_image to load files. (READ ONLY)
 // If you enable SDL_image all other libraries will be ignored!
 {.$DEFINE GLB_SDL_IMAGE}
@@ -286,16 +286,16 @@ unit glBitmap;
 {$ENDIF}
 
 // Operation System
-{$IF DEFINED(WIN32) or DEFINED(WIN64)}
+{$IF DEFINED(WIN32) or DEFINED(WIN64) or DEFINED(WINDOWS)}
   {$DEFINE GLB_WIN}
 {$ELSEIF DEFINED(LINUX)}
   {$DEFINE GLB_LINUX}
-{$ENDIF}
+{$IFEND}
 
 // native OpenGL Support
 {$IF DEFINED(GLB_NATIVE_OGL_STATIC) OR DEFINED(GLB_NATIVE_OGL_DYNAMIC)}
   {$DEFINE GLB_NATIVE_OGL}
-{$ENDIF}
+{$IFEND}
 
 // checking define combinations
 //SDL Image
@@ -362,7 +362,7 @@ unit glBitmap;
 // native OpenGL
 {$IF DEFINED(GLB_NATIVE_OGL_STATIC) AND DEFINED(GLB_NATIVE_OGL_DYNAMIC)}
   {$MESSAGE warn 'GLB_NATIVE_OGL_STATIC will be ignored because you enabled GLB_NATIVE_OGL_DYNAMIC'}
-{$ENDIF}
+{$IFEND}
 
 // general options
 {$EXTENDEDSYNTAX ON}
@@ -377,7 +377,7 @@ interface
 uses
   {$IFNDEF GLB_NATIVE_OGL}      dglOpenGL,          {$ENDIF}
   {$IF DEFINED(GLB_WIN) AND
-       DEFINED(GLB_NATIVE_OGL)} windows,            {$ENDIF}
+       DEFINED(GLB_NATIVE_OGL)} windows,            {$IFEND}
 
   {$IFDEF GLB_SDL}              SDL,                {$ENDIF}
   {$IFDEF GLB_LAZARUS}          IntfGraphics, GraphType,       {$ENDIF}
@@ -393,20 +393,6 @@ uses
 
   Classes, SysUtils;
 
-{$IFNDEF GLB_DELPHI}
-type
-  HGLRC = Cardinal;
-  DWORD = Cardinal;
-  PDWORD = ^DWORD;
-
-  TRGBQuad = packed record
-    rgbBlue: Byte;
-    rgbGreen: Byte;
-    rgbRed: Byte;
-    rgbReserved: Byte;
-  end;
-{$ENDIF}
-
 {$IFDEF GLB_NATIVE_OGL}
 const
   GL_TRUE   = 1;
@@ -524,13 +510,13 @@ const
   GL_TEXTURE_MAX_ANISOTROPY_EXT     = $84FE;
   GL_MAX_TEXTURE_MAX_ANISOTROPY_EXT = $84FF;
 
-{$ifdef LINUX}
-  libglu    = 'libGLU.so.1';
-  libopengl = 'libGL.so.1';
-{$else}
+{$IF DEFINED(GLB_WIN)}
   libglu    = 'glu32.dll';
   libopengl = 'opengl32.dll';
-{$endif}
+{$ELSEIF DEFINED(GLB_LINUX)}
+  libglu    = 'libGLU.so.1';
+  libopengl = 'libGL.so.1';
+{$IFEND}
 
 type
   GLboolean = BYTEBOOL;
@@ -550,6 +536,13 @@ type
   TglCompressedTexImage2D  = procedure(target: GLenum; level: GLint; internalformat: GLenum; width: GLsizei; height: GLsizei; border: GLint; imageSize: GLsizei; const data: PGLvoid); {$IFDEF GLB_WIN}stdcall; {$ELSE}cdecl; {$ENDIF}
   TglGetCompressedTexImage = procedure(target: GLenum; level: GLint; img: PGLvoid); {$IFDEF GLB_WIN}stdcall; {$ELSE}cdecl; {$ENDIF}
 
+{$IF DEFINED(GLB_WIN)}
+  TwglGetProcAddress = function (ProcName: PAnsiChar): Pointer; stdcall;
+{$ELSEIF DEFINED(GLB_LINUX)}
+  TglXGetProcAddress = function(ProcName: PAnsiChar): Pointer; cdecl;
+  TglXGetProcAddressARB = function(const name: PAnsiChar): pointer; cdecl;
+{$IFEND}
+
 {$IF DEFINED(GLB_NATIVE_OGL_DYNAMIC)}
   TglEnable  = procedure(cap: GLenum); {$IFDEF GLB_WIN}stdcall; {$ELSE}cdecl; {$ENDIF}
   TglDisable = procedure(cap: GLenum); {$IFDEF GLB_WIN}stdcall; {$ELSE}cdecl; {$ENDIF}
@@ -579,13 +572,6 @@ type
   TgluBuild1DMipmaps = function(target: GLEnum; components, width: GLint; format, atype: GLEnum; const data: Pointer): GLint; {$IFDEF GLB_WIN}stdcall; {$ELSE}cdecl; {$ENDIF}
   TgluBuild2DMipmaps = function(target: GLEnum; components, width, height: GLint; format, atype: GLEnum; const Data: Pointer): GLint; {$IFDEF GLB_WIN}stdcall; {$ELSE}cdecl; {$ENDIF}
 
-  {$IFDEF GLB_LINUX}
-  TglXGetProcAddress = function(ProcName: PAnsiChar): Pointer; cdecl;
-  TglXGetProcAddressARB = function(const name: PAnsiChar): pointer; cdecl;
-  {$ELSE}
-  TwglGetProcAddress = function(ProcName: PAnsiChar): Pointer; stdcall;
-  {$ENDIF}
-
 {$ELSEIF DEFINED(GLB_NATIVE_OGL_STATIC)}
   procedure glEnable(cap: GLenum); {$IFDEF GLB_WIN}stdcall; {$ELSE}cdecl; {$ENDIF} external libopengl;
   procedure glDisable(cap: GLenum); {$IFDEF GLB_WIN}stdcall; {$ELSE}cdecl; {$ENDIF} external libopengl;
@@ -614,7 +600,7 @@ type
 
   function gluBuild1DMipmaps(target: GLEnum; components, width: GLint; format, atype: GLEnum; const data: Pointer): GLint; {$IFDEF GLB_WIN}stdcall; {$ELSE}cdecl; {$ENDIF} external libglu;
   function gluBuild2DMipmaps(target: GLEnum; components, width, height: GLint; format, atype: GLEnum; const Data: Pointer): GLint; {$IFDEF GLB_WIN}stdcall; {$ELSE}cdecl; {$ENDIF} external libglu;
-{$ENDIF}
+{$IFEND}
 
 var
   GL_VERSION_1_2,
@@ -641,6 +627,13 @@ var
   glCompressedTexImage2D: TglCompressedTexImage2D;
   glGetCompressedTexImage: TglGetCompressedTexImage;
 
+{$IF DEFINED(GLB_WIN)}
+  wglGetProcAddress: TwglGetProcAddress;
+{$ELSEIF DEFINED(GLB_LINUX)}
+  glXGetProcAddress: TglXGetProcAddress;
+  glXGetProcAddressARB: TglXGetProcAddress;
+{$IFEND}
+
 {$IFDEF GLB_NATIVE_OGL_DYNAMIC}
   glEnable: TglEnable;
   glDisable: TglDisable;
@@ -669,22 +662,7 @@ var
 
   gluBuild1DMipmaps: TgluBuild1DMipmaps;
   gluBuild2DMipmaps: TgluBuild2DMipmaps;
-
-  {$IF DEFINED(GLB_WIN)}
-  wglGetProcAddress: TwglGetProcAddress;
-  {$ELSEIF DEFINED(GLB_LINUX)}
-  glXGetProcAddress: TglXGetProcAddress;
-  glXGetProcAddressARB: TglXGetProcAddressARB;
-  {$ENDIF}
 {$ENDIF}
-
-(*
-{$IFDEF GLB_DELPHI}
-var
-  gLastContext: HGLRC;
-{$ENDIF}
-*)
-
 {$ENDIF}
 
 type
@@ -868,8 +846,8 @@ type
 
     procedure CreateID;
     procedure SetupParameters(out aBuildWithGlu: Boolean);
-    procedure SetDataPointer(const aData: PByte; const aFormat: TglBitmapFormat;
-      const aWidth: Integer = -1; const aHeight: Integer = -1); virtual;
+    procedure SetDataPointer(var aData: PByte; const aFormat: TglBitmapFormat;
+      const aWidth: Integer = -1; const aHeight: Integer = -1); virtual;  //be careful, aData could be freed by this method
     procedure GenTexture(const aTestTextureSize: Boolean = true); virtual; abstract;
 
     function FlipHorz: Boolean; virtual;
@@ -1015,15 +993,26 @@ type
   end;
 
 //////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////
+  TglBitmap1D = class(TglBitmap)
+  protected
+    procedure SetDataPointer(var aData: PByte; const aFormat: TglBitmapFormat;
+      const aWidth: Integer = - 1; const aHeight: Integer = - 1); override;
+    procedure UploadData(const aBuildWithGlu: Boolean);
+  public
+    property Width;
+    procedure AfterConstruction; override;
+    function FlipHorz: Boolean; override;
+    procedure GenTexture(const aTestTextureSize: Boolean = true); override;
+  end;
+
+//////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////
   TglBitmap2D = class(TglBitmap)
   protected
-    // Bildeinstellungen
     fLines: array of PByte;
-
     function GetScanline(const aIndex: Integer): Pointer;
-    procedure SetDataPointer(const aData: PByte; const aFormat: TglBitmapFormat;
+    procedure SetDataPointer(var aData: PByte; const aFormat: TglBitmapFormat;
       const aWidth: Integer = - 1; const aHeight: Integer = - 1); override;
-    procedure UploadData(const aTarget: Cardinal; const aBuildWithGlu: Boolean);
+    procedure UploadData(const aTarget: GLenum; const aBuildWithGlu: Boolean);
   public
     property Width;
     property Height;
@@ -1042,51 +1031,25 @@ type
       const aScale: Single = 2; const aUseAlpha: Boolean = false);
   end;
 
-(* TODO
+//////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////
   TglBitmapCubeMap = class(TglBitmap2D)
   protected
     fGenMode: Integer;
-
-    // Hide GenTexture
-    procedure GenTexture(TestTextureSize: Boolean = true); reintroduce;
+    procedure GenTexture(const aTestTextureSize: Boolean = true); reintroduce;
   public
     procedure AfterConstruction; override;
-
-    procedure GenerateCubeMap(CubeTarget: Cardinal; TestTextureSize: Boolean = true);
-
-    procedure Unbind(DisableTexCoordsGen: Boolean = true; DisableTextureUnit: Boolean = true); reintroduce; virtual;
-    procedure Bind(EnableTexCoordsGen: Boolean = true; EnableTextureUnit: Boolean = true); reintroduce; virtual;
+    procedure GenerateCubeMap(const aCubeTarget: Cardinal; const aTestTextureSize: Boolean = true);
+    procedure Bind(const aEnableTexCoordsGen: Boolean = true; const aEnableTextureUnit: Boolean = true); reintroduce; virtual;
+    procedure Unbind(const aDisableTexCoordsGen: Boolean = true; const aDisableTextureUnit: Boolean = true); reintroduce; virtual;
   end;
 
-
+//////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////
   TglBitmapNormalMap = class(TglBitmapCubeMap)
   public
     procedure AfterConstruction; override;
-
-    procedure GenerateNormalMap(Size: Integer = 32; TestTextureSize: Boolean = true);
+    procedure GenerateNormalMap(const aSize: Integer = 32; const aTestTextureSize: Boolean = true);
   end;
 
-
-  TglBitmap1D = class(TglBitmap)
-  protected
-    procedure GetPixel1DUnmap(const Pos: TglBitmapPixelPosition; var Pixel: TglBitmapPixelData);
-
-    procedure SetDataPointer(Data: pByte; Format: TglBitmapInternalFormat; Width: Integer = -1; Height: Integer = -1); override;
-    procedure UploadData (Target, Format, InternalFormat, Typ: Cardinal; BuildWithGlu: Boolean);
-  public
-    // propertys
-    property Width;
-
-    procedure AfterConstruction; override;
-
-    // Other
-    function FlipHorz: Boolean; override;
-
-    // Generation
-    procedure GenTexture(TestTextureSize: Boolean = true); override;
-  end;
-*)
-
 const
   NULL_SIZE: TglBitmapPixelPosition = (Fields: []; X: 0; Y: 0);
 
@@ -1132,6 +1095,14 @@ uses
   Math, syncobjs, typinfo;
 
 type
+{$IFNDEF fpc}
+  QWord   = System.UInt64;
+  PQWord  = ^QWord;
+
+  PtrInt  = Longint;
+  PtrUInt = DWord;
+{$ENDIF}
+
 ////////////////////////////////////////////////////////////////////////////////////////////////////
   TShiftRec = packed record
   case Integer of
@@ -1186,8 +1157,8 @@ type
     procedure Map(const aPixel: TglBitmapPixelData; var aData: PByte; var aMapData: Pointer); virtual; abstract;
     procedure Unmap(var aData: PByte; out aPixel: TglBitmapPixelData; var aMapData: Pointer); virtual; abstract;
 
-    function GetSize(const aSize: TglBitmapPixelPosition): Integer; virtual; overload;
-    function GetSize(const aWidth, aHeight: Integer): Integer; virtual; overload;
+    function GetSize(const aSize: TglBitmapPixelPosition): Integer; overload; virtual;
+    function GetSize(const aWidth, aHeight: Integer): Integer; overload; virtual; 
 
     function CreateMappingData: Pointer; virtual;
     procedure FreeMappingData(var aMappingData: Pointer); virtual;
@@ -1842,8 +1813,6 @@ var
 
 function glbGetProcAddress(aProcName: PChar; aLibHandle: Pointer = nil): Pointer;
 begin
-  result :=  nil;
-
   if not Assigned(aLibHandle) then
     aLibHandle := GL_LibHandle;
 
@@ -1868,7 +1837,7 @@ begin
   end;
 
   result := dlsym(aLibHandle, aProcName);
-{$ENDIF}
+{$IFEND}
   if not Assigned(result) then
     raise EglBitmapException.Create('unable to load procedure form library: ' + aProcName);
 end;
@@ -1891,7 +1860,7 @@ procedure glbInitOpenGL;
     result := dlopen(Name, RTLD_LAZY);
     {$ELSE}
     result := nil;
-    {$ENDIF}
+    {$IFEND}
   end;
 
   ////////////////////////////////////////////////////////////////////////////////
@@ -1905,7 +1874,7 @@ procedure glbInitOpenGL;
     Result := FreeLibrary({%H-}HINST(aLibHandle));
     {$ELSEIF DEFINED(GLB_LINUX)}
     Result := dlclose(aLibHandle) = 0;
-    {$ENDIF}
+    {$IFEND}
   end;
 
 begin
@@ -1928,8 +1897,8 @@ begin
     wglGetProcAddress    := glbGetProcAddress('wglGetProcAddress');
   {$ELSEIF DEFINED(GLB_LINUX)}
     glXGetProcAddress    := glbGetProcAddress('glXGetProcAddress');
-    glXGetProcAddressARB := dglGetProcAddress('glXGetProcAddressARB');
-  {$ENDIF}
+    glXGetProcAddressARB := glbGetProcAddress('glXGetProcAddressARB');
+  {$IFEND}
 
     glEnable := glbGetProcAddress('glEnable');
     glDisable := glbGetProcAddress('glDisable');
@@ -1963,9 +1932,6 @@ end;
 /////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////
 procedure glbReadOpenGLExtensions;
 var
-  {$IFDEF GLB_DELPHI}
-  Context: HGLRC;
-  {$ENDIF}
   Buffer: AnsiString;
   MajorVersion, MinorVersion: Integer;
 
@@ -2024,99 +1990,54 @@ begin
   end;
 {$ENDIF}
 
-{$IFDEF GLB_DELPHI}
-  Context := wglGetCurrentContext;
-  if (Context <> gLastContext) then begin
-    gLastContext := Context;
-{$ENDIF}
-
-    // Version
-    Buffer := glGetString(GL_VERSION);
-    TrimVersionString(Buffer, MajorVersion, MinorVersion);
-
-    GL_VERSION_1_2 := false;
-    GL_VERSION_1_3 := false;
-    GL_VERSION_1_4 := false;
-    GL_VERSION_2_0 := false;
-    if MajorVersion = 1 then begin
-      if MinorVersion >= 2 then
-        GL_VERSION_1_2 := true;
-
-      if MinorVersion >= 3 then
-        GL_VERSION_1_3 := true;
+  // Version
+  Buffer := glGetString(GL_VERSION);
+  TrimVersionString(Buffer, MajorVersion, MinorVersion);
 
-      if MinorVersion >= 4 then
-        GL_VERSION_1_4 := true;
-    end else if MajorVersion >= 2 then begin
+  GL_VERSION_1_2 := false;
+  GL_VERSION_1_3 := false;
+  GL_VERSION_1_4 := false;
+  GL_VERSION_2_0 := false;
+  if MajorVersion = 1 then begin
+    if MinorVersion >= 2 then
       GL_VERSION_1_2 := true;
-      GL_VERSION_1_3 := true;
-      GL_VERSION_1_4 := true;
-      GL_VERSION_2_0 := true;
-    end;
-
-    // Extensions
-    Buffer := glGetString(GL_EXTENSIONS);
-    GL_ARB_texture_border_clamp       := CheckExtension('GL_ARB_texture_border_clamp');
-    GL_ARB_texture_non_power_of_two   := CheckExtension('GL_ARB_texture_non_power_of_two');
-    GL_ARB_texture_rectangle          := CheckExtension('GL_ARB_texture_rectangle');
-    GL_ARB_texture_mirrored_repeat    := CheckExtension('GL_ARB_texture_mirrored_repeat');
-    GL_EXT_texture_edge_clamp         := CheckExtension('GL_EXT_texture_edge_clamp');
-    GL_EXT_texture_filter_anisotropic := CheckExtension('GL_EXT_texture_filter_anisotropic');
-    GL_EXT_texture_rectangle          := CheckExtension('GL_EXT_texture_rectangle');
-    GL_NV_texture_rectangle           := CheckExtension('GL_NV_texture_rectangle');
-    GL_IBM_texture_mirrored_repeat    := CheckExtension('GL_IBM_texture_mirrored_repeat');
-    GL_SGIS_generate_mipmap           := CheckExtension('GL_SGIS_generate_mipmap');
-
-    if GL_VERSION_1_3 then begin
-      glCompressedTexImage1D  := glbGetProcAddress('glCompressedTexImage1D');
-      glCompressedTexImage2D  := glbGetProcAddress('glCompressedTexImage2D');
-      glGetCompressedTexImage := glbGetProcAddress('glGetCompressedTexImage');
-    end else begin
-      glCompressedTexImage1D  := glbGetProcAddress('glCompressedTexImage1DARB');
-      glCompressedTexImage2D  := glbGetProcAddress('glCompressedTexImage2DARB');
-      glGetCompressedTexImage := glbGetProcAddress('glGetCompressedTexImageARB');
-    end;
-{$IFDEF GLB_DELPHI}
-  end;
-{$ENDIF}
-end;
-{$ENDIF}
-
-(* TODO GLB_DELPHI
-{$IFDEF GLB_DELPHI}
-function CreateGrayPalette: HPALETTE;
-var
-  Idx: Integer;
-  Pal: PLogPalette;
-begin
-  GetMem(Pal, SizeOf(TLogPalette) + (SizeOf(TPaletteEntry) * 256));
-
-  Pal.palVersion := $300;
-  Pal.palNumEntries := 256;
 
-  {$IFOPT R+}
-    {$DEFINE GLB_TEMPRANGECHECK}
-    {$R-}
-  {$ENDIF}
+    if MinorVersion >= 3 then
+      GL_VERSION_1_3 := true;
 
-  for Idx := 0 to 256 - 1 do begin
-    Pal.palPalEntry[Idx].peRed   := Idx;
-    Pal.palPalEntry[Idx].peGreen := Idx;
-    Pal.palPalEntry[Idx].peBlue  := Idx;
-    Pal.palPalEntry[Idx].peFlags := 0;
+    if MinorVersion >= 4 then
+      GL_VERSION_1_4 := true;
+  end else if MajorVersion >= 2 then begin
+    GL_VERSION_1_2 := true;
+    GL_VERSION_1_3 := true;
+    GL_VERSION_1_4 := true;
+    GL_VERSION_2_0 := true;
+  end;
+
+  // Extensions
+  Buffer := glGetString(GL_EXTENSIONS);
+  GL_ARB_texture_border_clamp       := CheckExtension('GL_ARB_texture_border_clamp');
+  GL_ARB_texture_non_power_of_two   := CheckExtension('GL_ARB_texture_non_power_of_two');
+  GL_ARB_texture_rectangle          := CheckExtension('GL_ARB_texture_rectangle');
+  GL_ARB_texture_mirrored_repeat    := CheckExtension('GL_ARB_texture_mirrored_repeat');
+  GL_EXT_texture_edge_clamp         := CheckExtension('GL_EXT_texture_edge_clamp');
+  GL_EXT_texture_filter_anisotropic := CheckExtension('GL_EXT_texture_filter_anisotropic');
+  GL_EXT_texture_rectangle          := CheckExtension('GL_EXT_texture_rectangle');
+  GL_NV_texture_rectangle           := CheckExtension('GL_NV_texture_rectangle');
+  GL_IBM_texture_mirrored_repeat    := CheckExtension('GL_IBM_texture_mirrored_repeat');
+  GL_SGIS_generate_mipmap           := CheckExtension('GL_SGIS_generate_mipmap');
+
+  if GL_VERSION_1_3 then begin
+    glCompressedTexImage1D  := glbGetProcAddress('glCompressedTexImage1D');
+    glCompressedTexImage2D  := glbGetProcAddress('glCompressedTexImage2D');
+    glGetCompressedTexImage := glbGetProcAddress('glGetCompressedTexImage');
+  end else begin
+    glCompressedTexImage1D  := glbGetProcAddress('glCompressedTexImage1DARB');
+    glCompressedTexImage2D  := glbGetProcAddress('glCompressedTexImage2DARB');
+    glGetCompressedTexImage := glbGetProcAddress('glGetCompressedTexImageARB');
   end;
-
-  {$IFDEF GLB_TEMPRANGECHECK}
-    {$UNDEF GLB_TEMPRANGECHECK}
-    {$R+}
-  {$ENDIF}
-
-  result := CreatePalette(Pal^);
-
-  FreeMem(Pal);
 end;
 {$ENDIF}
-*)
 
 {$IFDEF GLB_SDL_IMAGE}
 /////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////
@@ -3798,7 +3719,7 @@ begin
         aData^ := 0;
       d := LuminanceWeight(aPixel) and Range.r;
       aData^ := aData^ or (d shl (4 - {%H-}PtrUInt(aMapData)));
-      inc(aMapData, 4);
+      inc(PByte(aMapData), 4);
       if ({%H-}PtrUInt(aMapData) >= 8) then begin
         inc(aData);
         aMapData := nil;
@@ -3848,10 +3769,10 @@ begin
     aPixel.Data.b := b;
     aPixel.Data.a := a;
   end;
-  inc(aMapData, bits);
+  inc(PByte(aMapData), bits);
   if ({%H-}PtrUInt(aMapData) >= 8) then begin
     inc(aData, 1);
-    dec(aMapData, 8);
+    dec(PByte(aMapData), 8);
   end;
   inc(aData, s);
 end;
@@ -4092,7 +4013,7 @@ begin
     exit;
   if TFormatDescriptor.Get(Format).PixelSize <> TFormatDescriptor.Get(aValue).PixelSize then
     raise EglBitmapUnsupportedFormat.Create(Format);
-  SetDataPointer(Data, aValue, Width, Height);
+  SetDataPointer(fData, aValue, Width, Height); //be careful, Data could be freed by this method
 end;
 
 //////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////
@@ -4178,7 +4099,7 @@ begin
 end;
 
 //////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////
-procedure TglBitmap.SetDataPointer(const aData: PByte; const aFormat: TglBitmapFormat;
+procedure TglBitmap.SetDataPointer(var aData: PByte; const aFormat: TglBitmapFormat;
   const aWidth: Integer; const aHeight: Integer);
 var
   s: Single;
@@ -4246,8 +4167,11 @@ end;
 
 //////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////
 procedure TglBitmap.BeforeDestruction;
+var
+  NewData: PByte;
 begin
-  SetDataPointer(nil, tfEmpty);
+  NewData := nil;
+  SetDataPointer(NewData, tfEmpty); //be careful, Data could be freed by this method
   if (fID > 0) and fDeleteTextureOnFree then
     glDeleteTextures(1, @fID);
   inherited BeforeDestruction;
@@ -4308,9 +4232,10 @@ begin
   GetMem(tmpData, size);
   try
     FillChar(tmpData^, size, #$FF);
-    SetDataPointer(tmpData, aFormat, aSize.X, aSize.Y);
+    SetDataPointer(tmpData, aFormat, aSize.X, aSize.Y); //be careful, Data could be freed by this method
   except
-    FreeMem(tmpData);
+    if Assigned(tmpData) then
+      FreeMem(tmpData);
     raise;
   end;
   AddFunc(Self, aFunc, false, Format, aArgs);
@@ -4448,7 +4373,7 @@ begin
 
         // Updating Image or InternalFormat
         if aCreateTemp then
-          SetDataPointer(TmpData, aFormat, aSource.Width, aSource.Height)
+          SetDataPointer(TmpData, aFormat, aSource.Width, aSource.Height) //be careful, Data could be freed by this method
         else if (aFormat <> fFormat) then
           Format := aFormat;
 
@@ -4458,7 +4383,7 @@ begin
         DestFD.FreeMappingData(DestMD);
       end;
     except
-      if aCreateTemp then
+      if aCreateTemp and Assigned(TmpData) then
         FreeMem(TmpData);
       raise;
     end;
@@ -4553,10 +4478,11 @@ begin
           Inc(pTempData, RowSize);
         end;
       end;
-      SetDataPointer(pData, IntFormat, TempWidth, TempHeight);
+      SetDataPointer(pData, IntFormat, TempWidth, TempHeight); //be careful, Data could be freed by this method
       result := true;
     except
-      FreeMem(pData);
+      if Assigned(pData) then
+        FreeMem(pData);
       raise;
     end;
   end;
@@ -4621,7 +4547,27 @@ end;
 {$ENDIF}
 
 {$IFDEF GLB_DELPHI}
-//TODO rework & test
+//////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////
+function CreateGrayPalette: HPALETTE;
+var
+  Idx: Integer;
+  Pal: PLogPalette;
+begin
+  GetMem(Pal, SizeOf(TLogPalette) + (SizeOf(TPaletteEntry) * 256));
+
+  Pal.palVersion := $300;
+  Pal.palNumEntries := 256;
+
+  for Idx := 0 to Pal.palNumEntries - 1 do begin
+    Pal.palPalEntry[Idx].peRed   := Idx;
+    Pal.palPalEntry[Idx].peGreen := Idx;
+    Pal.palPalEntry[Idx].peBlue  := Idx;
+    Pal.palPalEntry[Idx].peFlags := 0;
+  end;
+  Result := CreatePalette(Pal^);
+  FreeMem(Pal);
+end;
+
 //////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////
 function TglBitmap.AssignToBitmap(const aBitmap: TBitmap): Boolean;
 var
@@ -4635,35 +4581,30 @@ begin
       aBitmap.Height := Height;
 
       case Format of
-        tfAlpha8, ifLuminance, ifDepth8:
-          begin
-            Bitmap.PixelFormat := pf8bit;
-            Bitmap.Palette := CreateGrayPalette;
-          end;
-        ifRGB5A1:
-          Bitmap.PixelFormat := pf15bit;
-        ifR5G6B5:
-          Bitmap.PixelFormat := pf16bit;
-        ifRGB8, ifBGR8:
-          Bitmap.PixelFormat := pf24bit;
-        ifRGBA8, ifBGRA8:
-          Bitmap.PixelFormat := pf32bit;
-        else
-          raise EglBitmapException.Create('AssignToBitmap - Invalid Pixelformat.');
+        tfAlpha8, tfLuminance8: begin
+          aBitmap.PixelFormat := pf8bit;
+          aBitmap.Palette     := CreateGrayPalette;
+        end;
+        tfRGB5A1:
+          aBitmap.PixelFormat := pf15bit;
+        tfR5G6B5:
+          aBitmap.PixelFormat := pf16bit;
+        tfRGB8, tfBGR8:
+          aBitmap.PixelFormat := pf24bit;
+        tfRGBA8, tfBGRA8:
+          aBitmap.PixelFormat := pf32bit;
+      else
+        raise EglBitmapException.Create('AssignToBitmap - Invalid Pixelformat.');
       end;
 
       pSource := Data;
       for Row := 0 to FileHeight -1 do begin
-        pData := Bitmap.Scanline[Row];
-
+        pData := aBitmap.Scanline[Row];
         Move(pSource^, pData^, fRowSize);
         Inc(pSource, fRowSize);
-
-        // swap RGB(A) to BGR(A)
-        if InternalFormat in [ifRGB8, ifRGBA8] then
-          SwapRGB(pData, FileWidth, InternalFormat = ifRGBA8);
+        if (Format in [tfRGB8, tfRGBA8]) then        // swap RGB(A) to BGR(A)
+          SwapRGB(pData, FileWidth, Format = tfRGBA8);
       end;
-
       result := true;
     end;
   end;
@@ -4674,49 +4615,44 @@ function TglBitmap.AssignFromBitmap(const aBitmap: TBitmap): Boolean;
 var
   pSource, pData, pTempData: PByte;
   Row, RowSize, TempWidth, TempHeight: Integer;
-  IntFormat: TglBitmapInternalFormat;
+  IntFormat: TglBitmapFormat;
 begin
   result := false;
 
-  if (Assigned(Bitmap)) then begin
-    case Bitmap.PixelFormat of
+  if (Assigned(aBitmap)) then begin
+    case aBitmap.PixelFormat of
       pf8bit:
-        IntFormat := ifLuminance;
+        IntFormat := tfLuminance8;
       pf15bit:
-        IntFormat := ifRGB5A1;
+        IntFormat := tfRGB5A1;
       pf16bit:
-        IntFormat := ifR5G6B5;
+        IntFormat := tfR5G6B5;
       pf24bit:
-        IntFormat := ifBGR8;
+        IntFormat := tfBGR8;
       pf32bit:
-        IntFormat := ifBGRA8;
-      else
-        raise EglBitmapException.Create('AssignFromBitmap - Invalid Pixelformat.');
+        IntFormat := tfBGRA8;
+    else
+      raise EglBitmapException.Create('AssignFromBitmap - Invalid Pixelformat.');
     end;
 
-    TempWidth := Bitmap.Width;
-    TempHeight := Bitmap.Height;
-
-    RowSize := Trunc(TempWidth * FormatGetSize(IntFormat));
-
+    TempWidth  := aBitmap.Width;
+    TempHeight := aBitmap.Height;
+    RowSize    := TFormatDescriptor.Get(IntFormat).GetSize(TempWidth, 1);
     GetMem(pData, TempHeight * RowSize);
     try
       pTempData := pData;
-
       for Row := 0 to TempHeight -1 do begin
-        pSource := Bitmap.Scanline[Row];
-
+        pSource := aBitmap.Scanline[Row];
         if (Assigned(pSource)) then begin
           Move(pSource^, pTempData^, RowSize);
           Inc(pTempData, RowSize);
         end;
       end;
-
-      SetDataPointer(pData, IntFormat, TempWidth, TempHeight);
-
+      SetDataPointer(pData, IntFormat, TempWidth, TempHeight); //be careful, Data could be freed by this method
       result := true;
     except
-      FreeMem(pData);
+      if Assigned(pData) then
+        FreeMem(pData);
       raise;
     end;
   end;
@@ -4731,17 +4667,17 @@ begin
   result := false;
 
   if Assigned(Data) then begin
-    if InternalFormat in [ifAlpha, ifLuminanceAlpha, ifRGBA8, ifBGRA8] then begin
-      if Assigned(Bitmap) then begin
-        Bitmap.PixelFormat := pf8bit;
-        Bitmap.Palette := CreateGrayPalette;
-        Bitmap.Width := Width;
-        Bitmap.Height := Height;
-
-        case InternalFormat of
-          ifLuminanceAlpha:
+    if (Format in [tfAlpha8, tfLuminance8Alpha8, tfRGBA8, tfBGRA8]) then begin
+      if Assigned(aBitmap) then begin
+        aBitmap.PixelFormat := pf8bit;
+        aBitmap.Palette     := CreateGrayPalette;
+        aBitmap.Width       := Width;
+        aBitmap.Height      := Height;
+
+        case Format of
+          tfLuminance8Alpha8:
             AlphaInterleave := 1;
-          ifRGBA8, ifBGRA8:
+          tfRGBA8, tfBGRA8:
             AlphaInterleave := 3;
           else
             AlphaInterleave := 0;
@@ -4751,8 +4687,7 @@ begin
         pSource := Data;
 
         for Row := 0 to Height -1 do begin
-          pDest := Bitmap.Scanline[Row];
-
+          pDest := aBitmap.Scanline[Row];
           if Assigned(pDest) then begin
             for Col := 0 to Width -1 do begin
               Inc(pSource, AlphaInterleave);
@@ -4761,8 +4696,7 @@ begin
               Inc(pSource);
             end;
           end;
-        end;
-
+        end;   
         result := true;
       end;
     end;
@@ -4770,14 +4704,14 @@ begin
 end;
 
 //////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////
-function TglBitmap.AddAlphaFromBitmap(const aBitmap: TBitmap; const aFunc: TglBitmapFunction; const aArgs: PtrInt): Boolean;
+function TglBitmap.AddAlphaFromBitmap(const aBitmap: TBitmap; const aFunc: TglBitmapFunction; const aArgs: Pointer): Boolean;
 var
   tex: TglBitmap2D;
 begin
   tex := TglBitmap2D.Create;
   try
-    tex.AssignFromBitmap(Bitmap);
-    result := AddAlphaFromglBitmap(tex, Func, CustomData);
+    tex.AssignFromBitmap(ABitmap);
+    result := AddAlphaFromglBitmap(tex, aFunc, aArgs);
   finally
     tex.Free;
   end;
@@ -4864,9 +4798,10 @@ begin
   ImageData := GetMem(ImageSize);
   try
     Move(aImage.PixelData^, ImageData^, (aImage.Width * aImage.Height * aImage.DataDescription.BitsPerPixel) shr 3);
-    SetDataPointer(ImageData, f, aImage.Width, aImage.Height);
+    SetDataPointer(ImageData, f, aImage.Width, aImage.Height); //be careful, Data could be freed by this method
   except
-    FreeMem(ImageData);
+    if Assigned(ImageData) then
+      FreeMem(ImageData);
     raise;
   end;
 
@@ -5198,13 +5133,16 @@ begin
       GetMem(TempPtr, Size);
       try
         Move(Data^, TempPtr^, Size);
-        Temp.SetDataPointer(TempPtr, Format, Width, Height);
+        Temp.SetDataPointer(TempPtr, Format, Width, Height); //be careful, Data could be freed by this method
       except
-        FreeMem(TempPtr);
+        if Assigned(TempPtr) then
+          FreeMem(TempPtr);
         raise;
       end;
-    end else
-      Temp.SetDataPointer(nil, Format, Width, Height);
+    end else begin
+      TempPtr := nil;
+      Temp.SetDataPointer(TempPtr, Format, Width, Height); //be careful, Data could be freed by this method
+    end;
 
        // copy properties
     Temp.fID                      := ID;
@@ -5312,8 +5250,11 @@ end;
 
 //////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////
 procedure TglBitmap.FreeData;
+var
+  TempPtr: PByte;
 begin
-  SetDataPointer(nil, tfEmpty);
+  TempPtr := nil;
+  SetDataPointer(TempPtr, tfEmpty); //be careful, Data could be freed by this method
 end;
 
 //////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////
@@ -5471,11 +5412,11 @@ end;
 //////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////
 constructor TglBitmap.Create;
 begin
+  if (ClassType = TglBitmap) then
+    raise EglBitmapException.Create('Don''t create TglBitmap directly. Use one of the deviated classes (TglBitmap2D) instead.');
 {$IFDEF GLB_NATIVE_OGL}
   glbReadOpenGLExtensions;
 {$ENDIF}
-  if (ClassType = TglBitmap) then
-    raise EglBitmapException.Create('Don''t create TglBitmap directly. Use one of the deviated classes (TglBitmap2D) instead.');
   inherited Create;
 end;
 
@@ -5504,9 +5445,10 @@ begin
   GetMem(Image, ImageSize);
   try
     FillChar(Image^, ImageSize, #$FF);
-    SetDataPointer(Image, aFormat, aSize.X, aSize.Y);
+    SetDataPointer(Image, aFormat, aSize.X, aSize.Y); //be careful, Data could be freed by this method
   except
-    FreeMem(Image);
+    if Assigned(Image) then
+      FreeMem(Image);
     raise;
   end;
 end;
@@ -5660,11 +5602,12 @@ begin
         SetLength(png_rows, 0);
 
         // set new data
-        SetDataPointer(png_data, Format, TempWidth, TempHeight);
+        SetDataPointer(png_data, Format, TempWidth, TempHeight); //be careful, Data could be freed by this method
 
         result := true;
       except
-        FreeMem(png_data);
+        if Assigned(png_data) then
+          FreeMem(png_data);
         raise;
       end;
     end;
@@ -5753,11 +5696,12 @@ begin
             raise EglBitmapException.Create ('LoadPng - Unsupported Colortype found.');
         end;
 
-        SetDataPointer(NewImage, PngFormat, Png.Header.Width, Png.Header.Height);
+        SetDataPointer(NewImage, PngFormat, Png.Header.Width, Png.Header.Height); //be careful, Data could be freed by this method
 
         result := true;
       except
-        FreeMem(NewImage);
+        if Assigned(NewImage) then
+          FreeMem(NewImage);
         raise;
       end;
     finally
@@ -5765,7 +5709,7 @@ begin
     end;
   end;
 end;
-{$ENDIF}
+{$IFEND}
 {$ENDIF}
 
 {$IFDEF GLB_SUPPORT_PNG_WRITE}
@@ -5931,7 +5875,7 @@ begin
     FreeAndNil(Png);
   end;
 end;
-{$ENDIF}
+{$IFEND}
 {$ENDIF}
 
 //////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////
@@ -6194,11 +6138,12 @@ begin
         // destroy decompression
         jpeg_destroy_decompress(@jpeg);
 
-        SetDataPointer(pImage, IntFormat, TempWidth, TempHeight);
+        SetDataPointer(pImage, IntFormat, TempWidth, TempHeight); //be careful, Data could be freed by this method
 
         result := true;
       except
-        FreeMem(pImage);
+        if Assigned(pImage) then
+          FreeMem(pImage);
         raise;
       end;
     end;
@@ -6219,9 +6164,9 @@ begin
   result := false;
 
   // reading first two bytes to test file and set cursor back to begin
-  StreamPos := Stream.Position;
-  Stream.Read(Temp[0], 2);
-  Stream.Position := StreamPos;
+  StreamPos := aStream.Position;
+  aStream.Read(Temp[0], 2);
+  aStream.Position := StreamPos;
 
   // if Bitmap then read file.
   if ((Temp[0] = $FF) and (Temp[1] = $D8)) then begin
@@ -6229,7 +6174,7 @@ begin
     try
       jpg := TJPEGImage.Create;
       try
-        jpg.LoadFromStream(Stream);
+        jpg.LoadFromStream(aStream);
         bmp.Assign(jpg);
         result := AssignFromBitmap(bmp);
       finally
@@ -6240,7 +6185,7 @@ begin
     end;
   end;
 end;
-{$ENDIF}
+{$IFEND}
 {$ENDIF}
 
 {$IFDEF GLB_SUPPORT_JPEG_WRITE}
@@ -6351,25 +6296,25 @@ end;
 
 {$ELSEIF DEFINED(GLB_DELPHI_JPEG)}
 //////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////
-procedure TglBitmap.SaveJPEG(Stream: TStream);
+procedure TglBitmap.SaveJPEG(const aStream: TStream);
 var
   Bmp: TBitmap;
   Jpg: TJPEGImage;
 begin
-  if not (ftJPEG in FormatGetSupportedFiles (InternalFormat)) then
-    raise EglBitmapUnsupportedInternalFormat.Create('SaveJpg - ' + UNSUPPORTED_INTERNAL_FORMAT);
+  if not (ftJPEG in FormatGetSupportedFiles(Format)) then
+    raise EglBitmapUnsupportedFormat.Create(Format);
 
   Bmp := TBitmap.Create;
   try
     Jpg := TJPEGImage.Create;
     try
       AssignToBitmap(Bmp);
-      if FInternalFormat in [ifAlpha, ifLuminance, ifDepth8] then begin
-        Jpg.Grayscale := true;
+      if (Format in [tfAlpha8, tfLuminance8]) then begin
+        Jpg.Grayscale   := true;
         Jpg.PixelFormat := jf8Bit;
       end;
       Jpg.Assign(Bmp);
-      Jpg.SaveToStream(Stream);
+      Jpg.SaveToStream(aStream);
     finally
       FreeAndNil(Jpg);
     end;
@@ -6377,7 +6322,7 @@ begin
     FreeAndNil(Bmp);
   end;
 end;
-{$ENDIF}
+{$IFEND}
 {$ENDIF}
 
 //////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////
@@ -6591,7 +6536,7 @@ begin
               inc(TmpData, wbLineSize);
             aStream.Read(PaddingBuff{%H-}, Padding);
           end;
-          SetDataPointer(ImageData, BmpFormat, Info.biWidth, abs(Info.biHeight));
+          SetDataPointer(ImageData, BmpFormat, Info.biWidth, abs(Info.biHeight)); //be careful, Data could be freed by this method
           result := true;
         finally
           if Assigned(LineBuf) then
@@ -6601,7 +6546,8 @@ begin
           FormatDesc.FreeMappingData(DestMD);
         end;
         except
-          FreeMem(ImageData);
+          if Assigned(ImageData) then
+            FreeMem(ImageData);
           raise;
         end;
       end else
@@ -6844,13 +6790,15 @@ const
   begin
     buf := nil;
     if (Counter.X.dir < 0) then
-      buf := GetMem(LineSize);
+      GetMem(buf, LineSize);
     try
       while (Counter.Y.low <> Counter.Y.high + counter.Y.dir) do begin
-        tmp1 := ImageData + (Counter.Y.low * LineSize); //pointer to LineStart
+        tmp1 := ImageData;
+        inc(tmp1, (Counter.Y.low * LineSize));          //pointer to LineStart
         if (Counter.X.dir < 0) then begin               //flip X
           aStream.Read(buf^, LineSize);
-          tmp2 := buf + LineSize - PixelSize;           //pointer to last pixel in line
+          tmp2 := buf;
+          inc(tmp2, LineSize - PixelSize);              //pointer to last pixel in line
           for i := 0 to Header.Width-1 do begin         //for all pixels in line
             for j := 0 to PixelSize-1 do begin          //for all bytes in pixel
               tmp1^ := tmp2^;
@@ -6881,9 +6829,10 @@ const
       if (LinePixelsRead >= Header.Width) then begin
         LinePixelsRead := 0;
         inc(Counter.Y.low, Counter.Y.dir);                //next line index
-        TmpData := ImageData + Counter.Y.low * LineSize;  //set line
+        TmpData := ImageData;
+        inc(TmpData, Counter.Y.low * LineSize);           //set line
         if (Counter.X.dir < 0) then                       //if x flipped then
-          TmpData := TmpData + LineSize - PixelSize;      //set last pixel
+          inc(TmpData, LineSize - PixelSize);             //set last pixel
       end;
     end;
 
@@ -6961,9 +6910,10 @@ const
 
     GetMem(Cache, CACHE_SIZE);
     try
-      TmpData := ImageData + Counter.Y.low * LineSize;  //set line
+      TmpData := ImageData;
+      inc(TmpData, Counter.Y.low * LineSize);           //set line
       if (Counter.X.dir < 0) then                       //if x flipped then
-        TmpData := TmpData + LineSize - PixelSize;      //set last pixel
+        inc(TmpData, LineSize - PixelSize);             //set last pixel
 
       repeat
         //read CommandByte
@@ -7012,6 +6962,7 @@ begin
       if Header.ImageID <> 0 then       // skip image ID
         aStream.Position := aStream.Position + Header.ImageID;
 
+      tgaFormat := tfEmpty;        
       case Header.Bpp of
          8: if IsGrayFormat then case (Header.ImageDesc and $F) of
                0: tgaFormat := tfLuminance8;
@@ -7076,10 +7027,11 @@ begin
             ReadCompressed;
         end;
 
-        SetDataPointer(ImageData, tgaFormat, Header.Width, Header.Height);
+        SetDataPointer(ImageData, tgaFormat, Header.Width, Header.Height); //be careful, Data could be freed by this method
         result := true;
       except
-        FreeMem(ImageData);
+        if Assigned(ImageData) then
+          FreeMem(ImageData);
         raise;
       end;
     finally
@@ -7156,7 +7108,7 @@ begin
 
   if Assigned(Converter) then begin
     LineSize := FormatDesc.GetSize(Width, 1);
-    LineBuf  := GetMem(LineSize);
+    GetMem(LineBuf, LineSize);
     SourceMD := FormatDesc.CreateMappingData;
     DestMD   := Converter.CreateMappingData;
     try
@@ -7364,7 +7316,8 @@ begin
         DestMD   := FormatDesc.CreateMappingData;
         try
           for y := 0 to Header.dwHeight-1 do begin
-            TmpData := NewImage + y * LineSize;
+            TmpData := NewImage;
+            inc(TmpData, y * LineSize);
             SrcData := RowData;
             aStream.Read(SrcData^, RowSize);
             for x := 0 to Header.dwWidth-1 do begin
@@ -7399,10 +7352,11 @@ begin
       end else
         raise EglBitmapException.Create('LoadDDS - unsupported Pixelformat found.');
 
-      SetDataPointer(NewImage, ddsFormat, Header.dwWidth, Header.dwHeight);
+      SetDataPointer(NewImage, ddsFormat, Header.dwWidth, Header.dwHeight); //be careful, Data could be freed by this method
       result := true;
     except
-      FreeMem(NewImage);
+      if Assigned(NewImage) then
+        FreeMem(NewImage);
       raise;
     end;
   finally
@@ -7468,6 +7422,112 @@ begin
 end;
 
 //////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////
+//TglBitmap1D/////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////
+//////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////
+procedure TglBitmap1D.SetDataPointer(var aData: PByte; const aFormat: TglBitmapFormat;
+  const aWidth: Integer; const aHeight: Integer);
+var
+  pTemp: pByte;
+  Size: Integer;
+begin
+  if (aHeight > 1) then begin
+    Size := TFormatDescriptor.Get(aFormat).GetSize(aWidth, 1);
+    GetMem(pTemp, Size);
+    try
+      Move(aData^, pTemp^, Size);
+      FreeMem(aData);
+      aData := nil;
+    except
+      FreeMem(pTemp);
+      raise;
+    end;
+  end else
+    pTemp := aData;
+  inherited SetDataPointer(pTemp, aFormat, aWidth);
+end;
+
+//////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////
+function TglBitmap1D.FlipHorz: Boolean;
+var
+  Col: Integer;
+  pTempDest, pDest, pSource: PByte;
+begin
+  result := inherited FlipHorz;
+  if Assigned(Data) and not TFormatDescriptor.Get(Format).IsCompressed then begin
+    pSource := Data;
+    GetMem(pDest, fRowSize);
+    try
+      pTempDest := pDest;
+      Inc(pTempDest, fRowSize);
+      for Col := 0 to Width-1 do begin
+        dec(pTempDest, fPixelSize); //dec before, because ptr is behind last byte of data
+        Move(pSource^, pTempDest^, fPixelSize);
+        Inc(pSource, fPixelSize);
+      end;
+      SetDataPointer(pDest, Format, Width); //be careful, Data could be freed by this method
+      result := true;
+    except
+      if Assigned(pDest) then
+        FreeMem(pDest);
+      raise;
+    end;
+  end;
+end;
+
+//////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////
+procedure TglBitmap1D.UploadData(const aBuildWithGlu: Boolean);
+var
+  FormatDesc: TFormatDescriptor;
+begin
+  // Upload data
+  FormatDesc := TFormatDescriptor.Get(Format);
+  if FormatDesc.IsCompressed then
+    glCompressedTexImage1D(Target, 0, FormatDesc.glInternalFormat, Width, 0, FormatDesc.GetSize(Width, 1), Data)
+  else if aBuildWithGlu then
+    gluBuild1DMipmaps(Target, FormatDesc.glInternalFormat, Width, FormatDesc.glFormat, FormatDesc.glDataFormat, Data)
+  else
+    glTexImage1D(Target, 0, FormatDesc.glInternalFormat, Width, 0, FormatDesc.glFormat, FormatDesc.glDataFormat, Data);
+
+  // Free Data
+  if (FreeDataAfterGenTexture) then
+    FreeData;
+end;
+
+//////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////
+procedure TglBitmap1D.GenTexture(const aTestTextureSize: Boolean);
+var
+  BuildWithGlu, TexRec: Boolean;
+  TexSize: Integer;
+begin
+  if Assigned(Data) then begin
+    // Check Texture Size
+    if (aTestTextureSize) then begin
+      glGetIntegerv(GL_MAX_TEXTURE_SIZE, @TexSize);
+
+      if (Width > TexSize) then
+        raise EglBitmapSizeToLargeException.Create('TglBitmap1D.GenTexture - The size for the texture is to large. It''s may be not conform with the Hardware.');
+
+      TexRec := (GL_ARB_texture_rectangle or GL_EXT_texture_rectangle or GL_NV_texture_rectangle) and
+                (Target = GL_TEXTURE_RECTANGLE_ARB);
+      if not (IsPowerOfTwo(Width) or GL_ARB_texture_non_power_of_two or GL_VERSION_2_0 or TexRec) then
+        raise EglBitmapNonPowerOfTwoException.Create('TglBitmap1D.GenTexture - Rendercontex dosn''t support non power of two texture.');
+    end;
+
+    CreateId;
+    SetupParameters(BuildWithGlu);
+    UploadData(BuildWithGlu);
+    glAreTexturesResident(1, @fID, @fIsResident);
+  end;
+end;
+
+//////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////
+procedure TglBitmap1D.AfterConstruction;
+begin
+  inherited;
+  Target := GL_TEXTURE_1D;
+end;
+
+//////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////
 //TglBitmap2D/////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////
 //////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////
 function TglBitmap2D.GetScanline(const aIndex: Integer): Pointer;
@@ -7479,7 +7539,7 @@ begin
 end;
 
 //////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////
-procedure TglBitmap2D.SetDataPointer(const aData: PByte; const aFormat: TglBitmapFormat;
+procedure TglBitmap2D.SetDataPointer(var aData: PByte; const aFormat: TglBitmapFormat;
   const aWidth: Integer; const aHeight: Integer);
 var
   Idx, LineWidth: Integer;
@@ -7504,7 +7564,7 @@ begin
 end;
 
 //////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////
-procedure TglBitmap2D.UploadData(const aTarget: Cardinal; const aBuildWithGlu: Boolean);
+procedure TglBitmap2D.UploadData(const aTarget: GLenum; const aBuildWithGlu: Boolean);
 var
   FormatDesc: TFormatDescriptor;
 begin
@@ -7512,7 +7572,7 @@ begin
 
   FormatDesc := TFormatDescriptor.Get(Format);
   if FormatDesc.IsCompressed then begin
-    glCompressedTexImage2D(Target, 0, FormatDesc.glInternalFormat, Width, Height, 0, FormatDesc.GetSize(fDimension), Data)
+    glCompressedTexImage2D(aTarget, 0, FormatDesc.glInternalFormat, Width, Height, 0, FormatDesc.GetSize(fDimension), Data)
   end else if aBuildWithGlu then begin
     gluBuild2DMipmaps(aTarget, FormatDesc.Components, Width, Height,
       FormatDesc.glFormat, FormatDesc.glDataFormat, Data)
@@ -7551,10 +7611,11 @@ begin
   try
     glPixelStorei(GL_PACK_ALIGNMENT, 1);
     glReadPixels(aLeft, aTop, w, h, FormatDesc.glFormat, FormatDesc.glDataFormat, Temp);
-    SetDataPointer(Temp, Format, w, h);
+    SetDataPointer(Temp, Format, w, h); //be careful, Data could be freed by this method
     FlipVert;
   except
-    FreeMem(Temp);
+    if Assigned(Temp) then
+      FreeMem(Temp);
     raise;
   end;
 end;
@@ -7592,9 +7653,10 @@ begin
       glGetCompressedTexImage(Target, 0, Temp)
     else
      glGetTexImage(Target, 0, FormatDesc.glInternalFormat, FormatDesc.glDataFormat, Temp);
-    SetDataPointer(Temp, IntFormat, TempWidth, TempHeight);
+    SetDataPointer(Temp, IntFormat, TempWidth, TempHeight); //be careful, Data could be freed by this method
   except
-    FreeMem(Temp);
+    if Assigned(Temp) then
+      FreeMem(Temp);
     raise;
   end;
 end;
@@ -7615,7 +7677,6 @@ begin
 
       PotTex := IsPowerOfTwo(Height) and IsPowerOfTwo(Width);
       TexRec := (GL_ARB_texture_rectangle or GL_EXT_texture_rectangle or GL_NV_texture_rectangle) and (Target = GL_TEXTURE_RECTANGLE);
-
       if not (PotTex or GL_ARB_texture_non_power_of_two or GL_VERSION_2_0 or TexRec) then
         raise EglBitmapNonPowerOfTwoException.Create('TglBitmap2D.GenTexture - Rendercontex dosn''t support non power of two texture.');
     end;
@@ -7650,10 +7711,11 @@ begin
           Dec(TempDestData, fPixelSize);
         end;
       end;
-      SetDataPointer(DestData, Format);
+      SetDataPointer(DestData, Format); //be careful, Data could be freed by this method
       result := true;
     except
-      FreeMem(DestData);
+      if Assigned(DestData) then
+        FreeMem(DestData);
       raise;
     end;
   end;
@@ -7677,10 +7739,11 @@ begin
         Dec(TempDestData, fRowSize);
         Inc(SourceData, fRowSize);
       end;
-      SetDataPointer(DestData, Format);
+      SetDataPointer(DestData, Format); //be careful, Data could be freed by this method
       result := true;
     except
-      FreeMem(DestData);
+      if Assigned(DestData) then
+        FreeMem(DestData);
       raise;
     end;
   end;
@@ -7909,147 +7972,15 @@ begin
   end;
 end;
 
-
-
-
-
-
-
-
-
-(*
-procedure TglBitmap1D.SetDataPointer(Data: pByte; Format: TglBitmapInternalFormat; Width, Height: Integer);
-var
-  pTemp: pByte;
-  Size: Integer;
-begin
-  if Height > 1 then begin
-    // extract first line of the data
-    Size := FormatGetImageSize(glBitmapPosition(Width), Format);
-    GetMem(pTemp, Size);
-
-    Move(Data^, pTemp^, Size);
-
-    FreeMem(Data);
-  end else
-    pTemp := Data;
-
-  // set data pointer
-  inherited SetDataPointer(pTemp, Format, Width);
-
-  if FormatIsUncompressed(Format) then begin
-    fUnmapFunc := FormatGetUnMapFunc(Format);
-    fGetPixelFunc := GetPixel1DUnmap;
-  end;
-end;
-
-
-procedure TglBitmap1D.GetPixel1DUnmap(const Pos: TglBitmapPixelPosition; var Pixel: TglBitmapPixelData);
-var
-  pTemp: pByte;
-begin
-  pTemp := Data;
-  Inc(pTemp, Pos.X * fPixelSize);
-
-  fUnmapFunc(pTemp, Pixel);
-end;
-
-
-function TglBitmap1D.FlipHorz: Boolean;
-var
-  Col: Integer;
-  pTempDest, pDest, pSource: pByte;
-begin
-  result := inherited FlipHorz;
-
-  if Assigned(Data) and FormatIsUncompressed(InternalFormat) then begin
-    pSource := Data;
-
-    GetMem(pDest, fRowSize);
-    try
-      pTempDest := pDest;
-
-      Inc(pTempDest, fRowSize);
-      for Col := 0 to Width -1 do begin
-        Move(pSource^, pTempDest^, fPixelSize);
-
-        Inc(pSource, fPixelSize);
-        Dec(pTempDest, fPixelSize);
-      end;
-
-      SetDataPointer(pDest, InternalFormat);
-
-      result := true;
-    finally
-      FreeMem(pDest);
-    end;
-  end;
-end;
-
-
-procedure TglBitmap1D.UploadData (Target, Format, InternalFormat, Typ: Cardinal; BuildWithGlu: Boolean);
-begin
-  // Upload data
-  if Self.InternalFormat in [ifDXT1, ifDXT3, ifDXT5] then
-    glCompressedTexImage1D(Target, 0, InternalFormat, Width, 0, Trunc(Width * FormatGetSize(Self.InternalFormat)), Data)
-  else
-
-  // Upload data
-  if BuildWithGlu then
-    gluBuild1DMipmaps(Target, InternalFormat, Width, Format, Typ, Data)
-  else
-    glTexImage1D(Target, 0, InternalFormat, Width, 0, Format, Typ, Data);
-
-  // Freigeben
-  if (FreeDataAfterGenTexture) then
-    FreeData;
-end;
-
-
-procedure TglBitmap1D.GenTexture(TestTextureSize: Boolean);
-var
-  BuildWithGlu, TexRec: Boolean;
-  glFormat, glInternalFormat, glType: Cardinal;
-  TexSize: Integer;
-begin
-  if Assigned(Data) then begin
-    // Check Texture Size
-    if (TestTextureSize) then begin
-      glGetIntegerv(GL_MAX_TEXTURE_SIZE, @TexSize);
-
-      if (Width > TexSize) then
-        raise EglBitmapSizeToLargeException.Create('TglBitmap1D.GenTexture - The size for the texture is to large. It''s may be not conform with the Hardware.');
-
-      TexRec := (GL_ARB_texture_rectangle or GL_EXT_texture_rectangle or GL_NV_texture_rectangle) and
-                (Target = GL_TEXTURE_RECTANGLE_ARB);
-
-      if not (IsPowerOfTwo (Width) or GL_ARB_texture_non_power_of_two or GL_VERSION_2_0 or TexRec) then
-        raise EglBitmapNonPowerOfTwoException.Create('TglBitmap1D.GenTexture - Rendercontex dosn''t support non power of two texture.');
-    end;
-
-    CreateId;
-
-    SetupParameters(BuildWithGlu);
-    SelectFormat(InternalFormat, glFormat, glInternalFormat, glType);
-
-    UploadData(Target, glFormat, glInternalFormat, glType, BuildWithGlu);
-
-    // Infos sammeln
-    glAreTexturesResident(1, @fID, @fIsResident);
-  end;
-end;
-
-
-procedure TglBitmap1D.AfterConstruction;
+//////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////
+//TglBitmapCubeMap////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////
+//////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////
+procedure TglBitmapCubeMap.GenTexture(const aTestTextureSize: Boolean);
 begin
-  inherited;
-
-  Target := GL_TEXTURE_1D;
+  Assert(false, 'TglBitmapCubeMap.GenTexture - Don''t call GenTextures directly.');
 end;
 
-
-{ TglBitmapCubeMap }
-
+//////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////
 procedure TglBitmapCubeMap.AfterConstruction;
 begin
   inherited;
@@ -8057,80 +7988,64 @@ begin
   if not (GL_VERSION_1_3 or GL_ARB_texture_cube_map or GL_EXT_texture_cube_map) then
     raise EglBitmapException.Create('TglBitmapCubeMap.AfterConstruction - CubeMaps are unsupported.');
 
-  SetWrap; // set all to GL_CLAMP_TO_EDGE
-  Target := GL_TEXTURE_CUBE_MAP;
+  SetWrap;
+  Target   := GL_TEXTURE_CUBE_MAP;
   fGenMode := GL_REFLECTION_MAP;
 end;
 
-
-procedure TglBitmapCubeMap.Bind(EnableTexCoordsGen, EnableTextureUnit: Boolean);
-begin
-  inherited Bind (EnableTextureUnit);
-
-  if EnableTexCoordsGen then begin
-    glTexGeni(GL_S, GL_TEXTURE_GEN_MODE, fGenMode);
-    glTexGeni(GL_T, GL_TEXTURE_GEN_MODE, fGenMode);
-    glTexGeni(GL_R, GL_TEXTURE_GEN_MODE, fGenMode);
-    glEnable(GL_TEXTURE_GEN_S);
-    glEnable(GL_TEXTURE_GEN_T);
-    glEnable(GL_TEXTURE_GEN_R);
-  end;
-end;
-
-
-procedure TglBitmapCubeMap.GenerateCubeMap(CubeTarget: Cardinal; TestTextureSize: Boolean);
+//////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////
+procedure TglBitmapCubeMap.GenerateCubeMap(const aCubeTarget: Cardinal; const aTestTextureSize: Boolean);
 var
-  glFormat, glInternalFormat, glType: Cardinal;
   BuildWithGlu: Boolean;
   TexSize: Integer;
 begin
-  // Check Texture Size
-  if (TestTextureSize) then begin
+  if (aTestTextureSize) then begin
     glGetIntegerv(GL_MAX_CUBE_MAP_TEXTURE_SIZE, @TexSize);
 
-    if ((Height > TexSize) or (Width > TexSize)) then
+    if (Height > TexSize) or (Width > TexSize) then
       raise EglBitmapSizeToLargeException.Create('TglBitmapCubeMap.GenTexture - The size for the Cubemap is to large. It''s may be not conform with the Hardware.');
 
-    if not ((IsPowerOfTwo (Height) and IsPowerOfTwo (Width)) or GL_VERSION_2_0 or GL_ARB_texture_non_power_of_two) then
+    if not ((IsPowerOfTwo(Height) and IsPowerOfTwo(Width)) or GL_VERSION_2_0 or GL_ARB_texture_non_power_of_two) then
       raise EglBitmapNonPowerOfTwoException.Create('TglBitmapCubeMap.GenTexture - Cubemaps dosn''t support non power of two texture.');
   end;
 
-  // create Texture
-  if ID = 0 then begin
+  if (ID = 0) then
     CreateID;
-    SetupParameters(BuildWithGlu);
-  end;
-
-  SelectFormat(InternalFormat, glFormat, glInternalFormat, glType);
-
-  UploadData (CubeTarget, glFormat, glInternalFormat, glType, BuildWithGlu);
+  SetupParameters(BuildWithGlu);
+  UploadData(aCubeTarget, BuildWithGlu);
 end;
 
-
-procedure TglBitmapCubeMap.GenTexture(TestTextureSize: Boolean);
+//////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////
+procedure TglBitmapCubeMap.Bind(const aEnableTexCoordsGen: Boolean; const aEnableTextureUnit: Boolean);
 begin
-  Assert(false, 'TglBitmapCubeMap.GenTexture - Don''t call GenTextures directly.');
+  inherited Bind (aEnableTextureUnit);
+  if aEnableTexCoordsGen then begin
+    glTexGeni(GL_S, GL_TEXTURE_GEN_MODE, fGenMode);
+    glTexGeni(GL_T, GL_TEXTURE_GEN_MODE, fGenMode);
+    glTexGeni(GL_R, GL_TEXTURE_GEN_MODE, fGenMode);
+    glEnable(GL_TEXTURE_GEN_S);
+    glEnable(GL_TEXTURE_GEN_T);
+    glEnable(GL_TEXTURE_GEN_R);
+  end;
 end;
 
-
-procedure TglBitmapCubeMap.Unbind(DisableTexCoordsGen,
-  DisableTextureUnit: Boolean);
+//////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////
+procedure TglBitmapCubeMap.Unbind(const aDisableTexCoordsGen: Boolean; const aDisableTextureUnit: Boolean);
 begin
-  inherited Unbind (DisableTextureUnit);
-
-  if DisableTexCoordsGen then begin
+  inherited Unbind(aDisableTextureUnit);
+  if aDisableTexCoordsGen then begin
     glDisable(GL_TEXTURE_GEN_S);
     glDisable(GL_TEXTURE_GEN_T);
     glDisable(GL_TEXTURE_GEN_R);
   end;
 end;
 
-
-{ TglBitmapNormalMap }
-
+//////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////
+//TglBitmapNormalMap//////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////
+//////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////
 type
   TVec = Array[0..2] of Single;
-  TglBitmapNormalMapGetVectorFunc = procedure (var Vec: TVec; const Position: TglBitmapPixelPosition; const HalfSize: Integer);
+  TglBitmapNormalMapGetVectorFunc = procedure (out aVec: TVec; const aPosition: TglBitmapPixelPosition; const aHalfSize: Integer);
 
   PglBitmapNormalMapRec = ^TglBitmapNormalMapRec;
   TglBitmapNormalMapRec = record
@@ -8138,62 +8053,63 @@ type
     Func: TglBitmapNormalMapGetVectorFunc;
   end;
 
-
-procedure glBitmapNormalMapPosX(var Vec: TVec; const Position: TglBitmapPixelPosition; const HalfSize: Integer);
+  //////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////
+procedure glBitmapNormalMapPosX(out aVec: TVec; const aPosition: TglBitmapPixelPosition; const aHalfSize: Integer);
 begin
-  Vec[0] := HalfSize;
-  Vec[1] := - (Position.Y + 0.5 - HalfSize);
-  Vec[2] := - (Position.X + 0.5 - HalfSize);
+  aVec[0] := aHalfSize;
+  aVec[1] := - (aPosition.Y + 0.5 - aHalfSize);
+  aVec[2] := - (aPosition.X + 0.5 - aHalfSize);
 end;
 
-
-procedure glBitmapNormalMapNegX(var Vec: TVec; const Position: TglBitmapPixelPosition; const HalfSize: Integer);
+//////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////
+procedure glBitmapNormalMapNegX(out aVec: TVec; const aPosition: TglBitmapPixelPosition; const aHalfSize: Integer);
 begin
-  Vec[0] := - HalfSize;
-  Vec[1] := - (Position.Y + 0.5 - HalfSize);
-  Vec[2] := Position.X + 0.5 - HalfSize;
+  aVec[0] := - aHalfSize;
+  aVec[1] := - (aPosition.Y + 0.5 - aHalfSize);
+  aVec[2] := aPosition.X + 0.5 - aHalfSize;
 end;
 
-
-procedure glBitmapNormalMapPosY(var Vec: TVec; const Position: TglBitmapPixelPosition; const HalfSize: Integer);
+//////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////
+procedure glBitmapNormalMapPosY(out aVec: TVec; const aPosition: TglBitmapPixelPosition; const aHalfSize: Integer);
 begin
-  Vec[0] := Position.X + 0.5 - HalfSize;
-  Vec[1] := HalfSize;
-  Vec[2] := Position.Y + 0.5 - HalfSize;
+  aVec[0] := aPosition.X + 0.5 - aHalfSize;
+  aVec[1] := aHalfSize;
+  aVec[2] := aPosition.Y + 0.5 - aHalfSize;
 end;
 
-
-procedure glBitmapNormalMapNegY(var Vec: TVec; const Position: TglBitmapPixelPosition; const HalfSize: Integer);
+//////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////
+procedure glBitmapNormalMapNegY(out aVec: TVec; const aPosition: TglBitmapPixelPosition; const aHalfSize: Integer);
 begin
-  Vec[0] := Position.X + 0.5 - HalfSize;
-  Vec[1] := - HalfSize;
-  Vec[2] := - (Position.Y + 0.5 - HalfSize);
+  aVec[0] := aPosition.X + 0.5 - aHalfSize;
+  aVec[1] := - aHalfSize;
+  aVec[2] := - (aPosition.Y + 0.5 - aHalfSize);
 end;
 
-
-procedure glBitmapNormalMapPosZ(var Vec: TVec; const Position: TglBitmapPixelPosition; const HalfSize: Integer);
+//////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////
+procedure glBitmapNormalMapPosZ(out aVec: TVec; const aPosition: TglBitmapPixelPosition; const aHalfSize: Integer);
 begin
-  Vec[0] := Position.X + 0.5 - HalfSize;
-  Vec[1] := - (Position.Y + 0.5 - HalfSize);
-  Vec[2] := HalfSize;
+  aVec[0] := aPosition.X + 0.5 - aHalfSize;
+  aVec[1] := - (aPosition.Y + 0.5 - aHalfSize);
+  aVec[2] := aHalfSize;
 end;
 
-
-procedure glBitmapNormalMapNegZ(var Vec: TVec; const Position: TglBitmapPixelPosition; const HalfSize: Integer);
+//////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////
+procedure glBitmapNormalMapNegZ(out aVec: TVec; const aPosition: TglBitmapPixelPosition; const aHalfSize: Integer);
 begin
-  Vec[0] := - (Position.X + 0.5 - HalfSize);
-  Vec[1] := - (Position.Y + 0.5 - HalfSize);
-  Vec[2] := - HalfSize;
+  aVec[0] := - (aPosition.X + 0.5 - aHalfSize);
+  aVec[1] := - (aPosition.Y + 0.5 - aHalfSize);
+  aVec[2] := - aHalfSize;
 end;
 
-
+//////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////
 procedure glBitmapNormalMapFunc(var FuncRec: TglBitmapFunctionRec);
 var
-  Vec : TVec;
+  i: Integer;
+  Vec: TVec;
   Len: Single;
 begin
   with FuncRec do begin
-    with PglBitmapNormalMapRec (CustomData)^ do begin
+    with PglBitmapNormalMapRec(Args)^ do begin
       Func(Vec, Position, HalfSize);
 
       // Normalize
@@ -8211,66 +8127,62 @@ begin
     end;
 
     // Set Color
-    Dest.Red   := Round(Vec[0] * 255);
-    Dest.Green := Round(Vec[1] * 255);
-    Dest.Blue  := Round(Vec[2] * 255);
+    for i := 0 to 2 do
+      Dest.Data.arr[i] := Round(Vec[i] * 255);
   end;
 end;
 
-
+//////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////
 procedure TglBitmapNormalMap.AfterConstruction;
 begin
   inherited;
-
   fGenMode := GL_NORMAL_MAP;
 end;
 
-
-procedure TglBitmapNormalMap.GenerateNormalMap(Size: Integer;
-  TestTextureSize: Boolean);
+//////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////
+procedure TglBitmapNormalMap.GenerateNormalMap(const aSize: Integer; const aTestTextureSize: Boolean);
 var
   Rec: TglBitmapNormalMapRec;
   SizeRec: TglBitmapPixelPosition;
 begin
-  Rec.HalfSize := Size div 2;
-
+  Rec.HalfSize := aSize div 2;
   FreeDataAfterGenTexture := false;
 
   SizeRec.Fields := [ffX, ffY];
-  SizeRec.X := Size;
-  SizeRec.Y := Size;
+  SizeRec.X := aSize;
+  SizeRec.Y := aSize;
 
   // Positive X
   Rec.Func := glBitmapNormalMapPosX;
-  LoadFromFunc (SizeRec, glBitmapNormalMapFunc, ifBGR8, @Rec);
-  GenerateCubeMap(GL_TEXTURE_CUBE_MAP_POSITIVE_X, TestTextureSize);
+  LoadFromFunc(SizeRec, glBitmapNormalMapFunc, tfBGR8, @Rec);
+  GenerateCubeMap(GL_TEXTURE_CUBE_MAP_POSITIVE_X, aTestTextureSize);
 
   // Negative X
   Rec.Func := glBitmapNormalMapNegX;
-  LoadFromFunc (SizeRec, glBitmapNormalMapFunc, ifBGR8, @Rec);
-  GenerateCubeMap(GL_TEXTURE_CUBE_MAP_NEGATIVE_X, TestTextureSize);
+  LoadFromFunc(SizeRec, glBitmapNormalMapFunc, tfBGR8, @Rec);
+  GenerateCubeMap(GL_TEXTURE_CUBE_MAP_NEGATIVE_X, aTestTextureSize);
 
   // Positive Y
   Rec.Func := glBitmapNormalMapPosY;
-  LoadFromFunc (SizeRec, glBitmapNormalMapFunc, ifBGR8, @Rec);
-  GenerateCubeMap(GL_TEXTURE_CUBE_MAP_POSITIVE_Y, TestTextureSize);
+  LoadFromFunc(SizeRec, glBitmapNormalMapFunc, tfBGR8, @Rec);
+  GenerateCubeMap(GL_TEXTURE_CUBE_MAP_POSITIVE_Y, aTestTextureSize);
 
   // Negative Y
   Rec.Func := glBitmapNormalMapNegY;
-  LoadFromFunc (SizeRec, glBitmapNormalMapFunc, ifBGR8, @Rec);
-  GenerateCubeMap(GL_TEXTURE_CUBE_MAP_NEGATIVE_Y, TestTextureSize);
+  LoadFromFunc(SizeRec, glBitmapNormalMapFunc, tfBGR8, @Rec);
+  GenerateCubeMap(GL_TEXTURE_CUBE_MAP_NEGATIVE_Y, aTestTextureSize);
 
   // Positive Z
   Rec.Func := glBitmapNormalMapPosZ;
-  LoadFromFunc (SizeRec, glBitmapNormalMapFunc, ifBGR8, @Rec);
-  GenerateCubeMap(GL_TEXTURE_CUBE_MAP_POSITIVE_Z, TestTextureSize);
+  LoadFromFunc(SizeRec, glBitmapNormalMapFunc, tfBGR8, @Rec);
+  GenerateCubeMap(GL_TEXTURE_CUBE_MAP_POSITIVE_Z, aTestTextureSize);
 
   // Negative Z
   Rec.Func := glBitmapNormalMapNegZ;
-  LoadFromFunc (SizeRec, glBitmapNormalMapFunc, ifBGR8, @Rec);
-  GenerateCubeMap(GL_TEXTURE_CUBE_MAP_NEGATIVE_Z, TestTextureSize);
+  LoadFromFunc(SizeRec, glBitmapNormalMapFunc, tfBGR8, @Rec);
+  GenerateCubeMap(GL_TEXTURE_CUBE_MAP_NEGATIVE_Z, aTestTextureSize);
 end;
-*)
+
 
 initialization
   glBitmapSetDefaultFormat(tfEmpty);