Use of ARB_fragment_program_100

[Phil Costin]

Hi,

This is marked as a TODO at line 1698 of dlls/wined3d/directx.c:

1691     if ((wined3d_settings.ps_mode == PS_HW) &&
GL_SUPPORT(ARB_FRAGMENT_PROGRAM) && (DeviceType != WINED3DDEVTYPE_REF)) {
1692         *pCaps->PixelShaderVersion    = D3DPS_VERSION(1,4);
1693         *pCaps->PixelShader1xMaxValue = 1.0;
1694     } else {
1695         *pCaps->PixelShaderVersion    = 0;
1696         *pCaps->PixelShader1xMaxValue = 0.0;
1697     }
1698     /* TODO: ARB_FRAGMENT_PROGRAM_100 */
1699 
1700     /* ------------------------------------------------
1701        The following fields apply to d3d9 only
1702        ------------------------------------------------ */
1703     if (This->dxVersion > 8) {

Despite remembering seeing this extension somewhere in the past, I can not
actually find any information relating to it by searching the web. I assume
that this is likely to need replacing with a
GL_SUPPORT(ARB_FRAGMENT_SHADER) as per line 1691 for pixel shaders > 1.4

There is probably a good reason for this line and I don't want to jump the
gun and mess with anyone's plans by submitting patches yet.

Are there definite plans to use this extension here? If so, is there any
documentation for it because I certainly can't find any...

Regards,
Phil Costin

[James Trotter]

On 4/23/06, Phil Costin  wrote:
> Hi,
>
> This is marked as a TODO at line 1698 of dlls/wined3d/directx.c:
>
> 1691     if ((wined3d_settings.ps_mode == PS_HW) &&
> GL_SUPPORT(ARB_FRAGMENT_PROGRAM) && (DeviceType != WINED3DDEVTYPE_REF)) {
> 1692         *pCaps->PixelShaderVersion    = D3DPS_VERSION(1,4);
> 1693         *pCaps->PixelShader1xMaxValue = 1.0;
> 1694     } else {
> 1695         *pCaps->PixelShaderVersion    = 0;
> 1696         *pCaps->PixelShader1xMaxValue = 0.0;
> 1697     }
> 1698     /* TODO: ARB_FRAGMENT_PROGRAM_100 */
> 1699
> 1700     /* ------------------------------------------------
> 1701        The following fields apply to d3d9 only
> 1702        ------------------------------------------------ */
> 1703     if (This->dxVersion > 8) {
>
> Despite remembering seeing this extension somewhere in the past, I can not
> actually find any information relating to it by searching the web. I
> assume
> that this is likely to need replacing with a
> GL_SUPPORT(ARB_FRAGMENT_SHADER) as per line 1691 for pixel shaders > 1.4
>
> There is probably a good reason for this line and I don't want to jump the
> gun and mess with anyone's plans by submitting patches yet.
>
> Are there definite plans to use this extension here? If so, is there any
> documentation for it because I certainly can't find any...
>
> Regards,
> Phil Costin
>
> On 4/23/06, Phil Costin  wrote:
>
> Hi,
>
> This is marked as a TODO at line 1698 of dlls/wined3d/directx.c:
>
> 1691     if ((wined3d_settings.ps_mode == PS_HW) &&
> GL_SUPPORT(ARB_FRAGMENT_PROGRAM) && (DeviceType != WINED3DDEVTYPE_REF)) {
> 1692         *pCaps->PixelShaderVersion    = D3DPS_VERSION(1,4);
> 1693         *pCaps->PixelShader1xMaxValue = 1.0;
> 1694     } else {
> 1695         *pCaps->PixelShaderVersion    = 0;
> 1696         *pCaps->PixelShader1xMaxValue = 0.0;
> 1697     }
> 1698     /* TODO: ARB_FRAGMENT_PROGRAM_100 */
> 1699
> 1700     /* ------------------------------------------------
> 1701        The following fields apply to d3d9 only
> 1702        ------------------------------------------------ */
> 1703     if (This->dxVersion > 8) {
>
> Despite remembering seeing this extension somewhere in the past, I can not
> actually find any information relating to it by searching the web. I
> assume
> that this is likely to need replacing with a
> GL_SUPPORT(ARB_FRAGMENT_SHADER) as per line 1691 for pixel shaders > 1.4
>
> There is probably a good reason for this line and I don't want to jump the
> gun and mess with anyone's plans by submitting patches yet.
>
> Are there definite plans to use this extension here? If so, is there any
> documentation for it because I certainly can't find any...
>
> Regards,
> Phil Costin

Hi.

I haven't looked at the code, but maybe it's the ARB_shading_language_100
extension that was intended?
(http://oss.sgi.com/projects/ogl-sample/registry/ARB/shading_language_100.txt
)

James D. Trotter

[Phil Costin]

James Trotter wrote:
> On 4/23/06, Phil Costin  wrote:
> > Hi,
> >
> > This is marked as a TODO at line 1698 of dlls/wined3d/directx.c:
> >
> > 1691     if ((wined3d_settings.ps_mode == PS_HW) &&
> > GL_SUPPORT(ARB_FRAGMENT_PROGRAM) && (DeviceType != WINED3DDEVTYPE_REF)) {
> > 1692         *pCaps->PixelShaderVersion    = D3DPS_VERSION(1,4);
> > 1693         *pCaps->PixelShader1xMaxValue = 1.0;
> > 1694     } else {
> > 1695         *pCaps->PixelShaderVersion    = 0;
> > 1696         *pCaps->PixelShader1xMaxValue = 0.0;
> > 1697     }
> > 1698     /* TODO: ARB_FRAGMENT_PROGRAM_100 */
> > 1699
> > 1700     /* ------------------------------------------------
> > 1701        The following fields apply to d3d9 only
> > 1702        ------------------------------------------------ */
> > 1703     if (This->dxVersion > 8) {
> >
> > Despite remembering seeing this extension somewhere in the past, I can
> > not actually find any information relating to it by searching the web. I
> > assume
> > that this is likely to need replacing with a
> > GL_SUPPORT(ARB_FRAGMENT_SHADER) as per line 1691 for pixel shaders > 1.4
> >
> > There is probably a good reason for this line and I don't want to jump
> > the gun and mess with anyone's plans by submitting patches yet.
> >
> > Are there definite plans to use this extension here? If so, is there any
> > documentation for it because I certainly can't find any...
> >
> > Regards,
> > Phil Costin
> >
> > On 4/23/06, Phil Costin  wrote:
> >
> > Hi,
> >
> > This is marked as a TODO at line 1698 of dlls/wined3d/directx.c:
> >
> > 1691     if ((wined3d_settings.ps_mode == PS_HW) &&
> > GL_SUPPORT(ARB_FRAGMENT_PROGRAM) && (DeviceType != WINED3DDEVTYPE_REF)) {
> > 1692         *pCaps->PixelShaderVersion    = D3DPS_VERSION(1,4);
> > 1693         *pCaps->PixelShader1xMaxValue = 1.0;
> > 1694     } else {
> > 1695         *pCaps->PixelShaderVersion    = 0;
> > 1696         *pCaps->PixelShader1xMaxValue = 0.0;
> > 1697     }
> > 1698     /* TODO: ARB_FRAGMENT_PROGRAM_100 */
> > 1699
> > 1700     /* ------------------------------------------------
> > 1701        The following fields apply to d3d9 only
> > 1702        ------------------------------------------------ */
> > 1703     if (This->dxVersion > 8) {
> >
> > Despite remembering seeing this extension somewhere in the past, I can
> > not actually find any information relating to it by searching the web. I
> > assume
> > that this is likely to need replacing with a
> > GL_SUPPORT(ARB_FRAGMENT_SHADER) as per line 1691 for pixel shaders > 1.4
> >
> > There is probably a good reason for this line and I don't want to jump
> > the gun and mess with anyone's plans by submitting patches yet.
> >
> > Are there definite plans to use this extension here? If so, is there any
> > documentation for it because I certainly can't find any...
> >
> > Regards,
> > Phil Costin
>
> Hi.
>
> I haven't looked at the code, but maybe it's the ARB_shading_language_100
> extension that was intended?

(http://oss.sgi.com/projects/ogl-sample/registry/ARB/shading_language_100.txt
> )
>
> James D. Trotter

Thanks James,

After some research earlier today, I also came to the same conclusion. I
have implemented a simple check for the GL_ARB_shading_language_100
extension (in the same way the other extensions are checked in directx.c)
in my tree and I'm testing it now. It might as well be submitted when AJ
returns from Greece. He may not include it because it doesn't support
anything yet, but it does get another piece of the caps code in place and
we will need this check anyway for when we decide at runtime whether to use
the existing ARBfvp1 or GLSL (which seems to be the route we're taking for
SM 2.0+).

Regards,
Phil